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

Update LICENSE #1211

Merged
merged 3 commits into from
Aug 20, 2024
Merged

Update LICENSE #1211

merged 3 commits into from
Aug 20, 2024

Conversation

clajo04
Copy link
Contributor

@clajo04 clajo04 commented Aug 16, 2024

The current LICENSE file does not support the GitHub License badge detecting the correct license of the project. Replacing the current license file with a copy of the license file from the Apache License 2.0 website will cause the GitHub License badge to correctly identify the license of jpype.

Running the licensee gem on the original repository resulted in the following output:

licensee detect
License: NOASSERTION
Matched files: LICENSE
LICENSE:
Content hash: c7acc21e9e359f098c0718f309ebd9d1fc15e355
License: NOASSERTION
Closest non-matching licenses:
Apache-2.0 similarity: 96.04%
ECL-2.0 similarity: 82.57%
OSL-3.0 similarity: 39.75%

Running the licensee gem on the updated repository results in the following output:

licensee detect
License: Apache-2.0
Matched files: LICENSE
LICENSE:
Content hash: bec905d850e7f5dc2e2db78a950d4a9db560a0b8
Confidence: 100.00%
Matcher: Licensee::Matchers::Exact
License: Apache-2.0

The current LICENSE file does not support the GitHub License badge detecting the correct license of the project.   Replacing the current license file with a copy of the license file from the Apache License 2.0 website will cause the GitHub License badge to correctly identify the license of jpype.
Copy link

codecov bot commented Aug 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.44%. Comparing base (653ccff) to head (98331a6).
Report is 160 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1211      +/-   ##
==========================================
+ Coverage   87.42%   87.44%   +0.01%     
==========================================
  Files         113      113              
  Lines       10238    10234       -4     
  Branches     4059     4059              
==========================================
- Hits         8951     8949       -2     
+ Misses        692      691       -1     
+ Partials      595      594       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Reversing out changes to the readme as I didn't realize my change on a local branch would submit to my PR on jpype-project.
@marscher marscher merged commit 7de66d0 into jpype-project:master Aug 20, 2024
10 of 20 checks passed
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.

2 participants