Skip to content

Avoid misinterpreting MIT license notice as Apache-2.0, issue #2635 - #2636

Merged
pombredanne merged 2 commits into
aboutcode-org:developfrom
petergardfjall:issue-2635
Aug 13, 2021
Merged

Avoid misinterpreting MIT license notice as Apache-2.0, issue #2635#2636
pombredanne merged 2 commits into
aboutcode-org:developfrom
petergardfjall:issue-2635

Conversation

@petergardfjall

@petergardfjall petergardfjall commented Aug 13, 2021

Copy link
Copy Markdown
Contributor

Fixes #2635

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 📁

This provides a rule that matches MIT notices of form (note: indent for highlight):

  License

  This project is licensed under the terms of the MIT license.

Signed-off-by: Peter Gardfjäll <peter.gardfjall.work@gmail.com>
This avoids Apache-2.0 "overriding" mit_349.RULE in cases like

  ## License

  This project is licensed under the terms of the MIT license.

Signed-off-by: Peter Gardfjäll <peter.gardfjall.work@gmail.com>
@petergardfjall petergardfjall changed the title Avoid misinterpreting MIT license notice as Apache-2.0 Issue #2635 Avoid misinterpreting MIT license notice as Apache-2.0, issue #2635 Aug 13, 2021

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Clean, neat and nice.
Thank you ++

@pombredanne
pombredanne merged commit 6bf2fd9 into aboutcode-org:develop Aug 13, 2021
@petergardfjall
petergardfjall deleted the issue-2635 branch August 16, 2021 05:51
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.

Apparent MIT license gets discovered as Apache-2.0

2 participants