Skip to content

LICENSE: tweak license file so the GitHub API detects it correctly. #10247

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

Merged
merged 1 commit into from
Jun 2, 2019

Conversation

rgommers
Copy link
Member

@rgommers rgommers commented Jun 2, 2019

Follow-up to gh-10235.

The GitHub API uses https://github.com/licensee/licensee. It's very sensitive. Now I've installed licensee locally and experimented until running this Ruby program prints out the desired result:

require "licensee"

scipy = Licensee.project("../scipy")

print scipy.license.name
print "\n"
print scipy.matched_file.filename
print "\n\n"

Result:

BSD 3-Clause "New" or "Revised" License
LICENSE.txt

Also note that licensee uses a cache based on the Git hash. So experimenting with it requires committing your change each time.

It uses https://github.com/licensee/licensee. It's very sensitive.
Now I've installed licensee locally and experimented until running
this Ruby program prints out the desired result:
```
require "licensee"

scipy = Licensee.project("../scipy")

print scipy.license.name
print "\n"
print scipy.matched_file.filename
print "\n\n"
```

Result:
```
BSD 3-Clause "New" or "Revised" License
LICENSE.txt
```

Also note that licensee uses a cache based on the Git hash. So
experimenting with it requires committing your change each time.
@rgommers
Copy link
Member Author

rgommers commented Jun 2, 2019

Okay I'm going to merge this, so I can verify that it actually resolved the issue. Please feel free to review and comment still, in case I missed something we can keep iterating till both everyone is happy and the GitHub API gives the desired result.

@rgommers rgommers merged commit 9164bbf into scipy:master Jun 2, 2019
@rgommers rgommers deleted the license branch June 2, 2019 11:44
@rgommers
Copy link
Member Author

rgommers commented Jun 2, 2019

Before:

image

After:

image

@WarrenWeckesser
Copy link
Member

Thanks Ralf!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Items related to regular maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants