Skip to content
This repository was archived by the owner on Jan 9, 2023. It is now read-only.

Conversation

@eduardo-rodrigues
Copy link
Member

@chrisburr, would you consider updating the info in the setup.py file as follows? Seems nicer to me for a community package.
I fixed the URL in any case.
There is also the issue of the license. I would push for a BSD-3 license as we have for all Scikit-HEP projects, where possible. Could you give that a thought? This would require some additions as a 1-line header to the various files, see the other projects ...
Let me know :-). Thanks.

@coveralls
Copy link

coveralls commented Feb 6, 2018

Coverage Status

Coverage remained the same at 91.346% when pulling 79b5bc6 on eduardo-setup-info into 42d6fbf on master.

@eduardo-rodrigues
Copy link
Member Author

Any comments @chrisburr ?

@chrisburr
Copy link
Member

I wasn't aware of the maintainer field but yes this is a nicer.

@ibab As this package is mostly your work, do you have any objections to it being BSD-3 instead of MIT to be more consistent with other scikit-hep packages?

@eduardo-rodrigues
Copy link
Member Author

Shall we just create an issue for the question about licensing and accept this PR? Seems the way to go :-).

@chrisburr chrisburr merged commit 7eff0fb into master Feb 23, 2018
@chrisburr
Copy link
Member

Yes!

@chrisburr chrisburr deleted the eduardo-setup-info branch February 23, 2018 23:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants