Skip to content
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

Fix license detection #64

Merged
merged 1 commit into from
May 16, 2024
Merged

Fix license detection #64

merged 1 commit into from
May 16, 2024

Conversation

leoheitmannruiz
Copy link
Contributor

Hey :)

Just a minor thing.

Currently licensee, the program used by GitHub to detect licenses, is not confident enough about what license is used:

License:        NOASSERTION
Matched files:  LICENSE
LICENSE:
  Content hash:  5227778a1739d9dabbcc2778eb28743ac1a96b8d
  License:       NOASSERTION
  Closest non-matching licenses:
    BSD-2-Clause similarity:  93.90%
    BSD-3-Clause similarity:  75.47%
    BSD-4-Clause similarity:  63.69%

This is because the disclaimer is slightly different from the one published by SPDX.

Now:

License:        BSD-2-Clause
Matched files:  LICENSE
LICENSE:
  Content hash:  a307ca750c21f262c95d2bce55e69c05e6c3dee9
  Attribution:   Copyright 2015-2019 Homebrew contributors.
  Confidence:    100.00%
  Matcher:       Licensee::Matchers::Exact
  License:       BSD-2-Clause

@MikeMcQuaid
Copy link
Member

Thanks again @leoheitmannruiz!

@MikeMcQuaid MikeMcQuaid enabled auto-merge May 16, 2024 02:35
@MikeMcQuaid MikeMcQuaid merged commit dc32dfc into Homebrew:master May 16, 2024
1 check passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants