-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Go modules support? #53
Comments
|
I built a go license tool for kubeflow.org in I took the approach described in this issue:
it doesn't automate as much as this tool did, because
but maybe we can improve from there taking sth already built here. |
This comment has been minimized.
This comment has been minimized.
Nevermind, I just tried putting a module into GOPATH and run However, urls are missing as stated
So what I suggested above can already be achieved by this tool. |
Hmmm, not exactly. I noticed a dangerous behavior, some repos do not have a license file. e.g. license file is embedded in README: https://github.com/upper/db/tree/v3.0.0. However, this tool does not give a warning of a module missing license about it when used in this way. |
I tried to implement ideas in this issue as https://github.com/Bobgy/go-mod-licenses to suit my own use-cases. |
Hey Bobgy, can you explain a little bit more in detail what you achieved? I just want to license check my local project's dependencies from My approach would be to:
|
@mreiche I've got a bit more requirements so did a few more complex things:
I've implemented a tool like described above and used it to generate licenses, example:
I want to open source my tool, but I'm currently fairly busy with other stuff. Might not have time to do that soon. Curious if these sound useful to others. |
Implemented in #94 |
UPDATE: I sent go-licenses/v2 proposal that takes go modules as first class citizen #70
Is there any plan for go modules support?
I'd like to get licenses for certain version of a library, because transitive dependency can be different between versions.
I believe this should address #33.The text was updated successfully, but these errors were encountered: