Skip to content

Conversation

@leoreinmann
Copy link
Contributor

@leoreinmann leoreinmann commented Jun 3, 2025

Fixes #4415

The key "code-credit-license-1.0-0" seems to be a typo because in the "short_name", the "name "and also the license text the version number is "1.0.0 ". The other licenses code-credit-license-1.0.1. and code-credit-license-1.1.0 have the semantic versioning pattern. Also on their website all their licenses use semantic versioning.

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @leoreinmann

Could you also chnage the license filename to fix the failing tests?
src/licensedcode/data/licenses/code-credit-license-1.0-0.LICENSE -> src/licensedcode/data/licenses/code-credit-license-1.0.0.LICENSE

@leoreinmann
Copy link
Contributor Author

@AyanSinhaMahapatra Yes, I've noticed it and changed it with the second commit 😄

@AyanSinhaMahapatra
Copy link
Member

Actually on further thought, we usually deprecate the license if the license key has a bug, and create a new license with the correct key and filename (with the same data verbatim) because:

  • this preserves the link to the license/licenseDB and the license key, since there are still people using old scancode versions where this could be detected.

@pombredanne what do you think?

@DennisClark
Copy link
Member

@AyanSinhaMahapatra and @pombredanne i think it's probably ok to update the license key in this case; it's a license that is very likely not used.

@leoreinmann
Copy link
Contributor Author

@AyanSinhaMahapatra Are the failing tests related to the PR?

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@leoreinmann no the test failures are related to #4369,
this is all good to merge now. Thanks++

@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 0c7022e into aboutcode-org:develop Jun 11, 2025
30 of 38 checks passed
alok1304 pushed a commit to alok1304/scancode-toolkit that referenced this pull request Jun 19, 2025
* fix: change version number in field "key" for code-credit-license-1.0-0
* fix: change file name for license "code-credit-license-1.0.0"

Signed-off-by: Leo Reinmann <[email protected]>
Signed-off-by: Alok Kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change license key for code-credit-license-1.0-0 to code-credit-license-1.0.0

3 participants