-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update licenses in license.go #57
Conversation
Sorry for all those commits. I found some edge cases that I had to fix. |
Hi, Line 1167 in 9af1752
|
@elrayle told me there are scripts in this repo for regenerating the license list. I'd like for her to weigh in here. |
Yep, I will take a look at this shortly. @darsh12 Thanks for pushing on this. The licenses need an update. FYI, the scripts to update licenses are in the cmd directory. See cmd/docs.md for instructions on how to run. The process for updating license ranges is manual. 😞 The rest of this is a reminder to myself... I see two short term changes that need to happen in response to your work.
And longer term...
NOTE: License ranges are hard because there isn't a consistent pattern to how they are grouped. |
Should the .json files be upgraded in https://github.com/github/go-spdx/tree/main/cmd too ? the ones which these updates are based on? Although not sure what the value of json files is, relative the spdx upstream git repo. |
PR #62 is very close to complete. It will run a nightly cron to auto-generate the code for getting licenses. The generated code will go in to a branch that can be more easily merged. Unfortunately, at least at this point, the range data still has to be manually updated. I intend to revisit this implementation to see if there is a way to at least simplify the handling of ranges. That will not be part of the auto-gen PR. I am hoping to resolve the remaining issues with the PR by Monday |
Thanks again for putting up this PR. Gave me the push I needed to put the nightly cron in place. Closing this PR as release v2.3.0 updates licenses to the latest version at this time which is SPDX License List v3.24.0 released on 2024-05-22 |
Upon using the ExtractLicense function, I found out that the license list is hard coded. After comparing the license list from spdx to the from
license.go
I found out many licenses that were missing. Hence, I came up with an automated approach to update all the various licenses being used.Also update one of the tests in license_test.go since the location of that test got updated
P,S. Happy to share my n8n workflow.