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

Improve/clean up packaging #176

Merged
merged 14 commits into from Apr 13, 2021
Merged

Improve/clean up packaging #176

merged 14 commits into from Apr 13, 2021

Conversation

jwodder
Copy link
Contributor

@jwodder jwodder commented Apr 13, 2021

See each commit message for explanations of the changes.

@coveralls
Copy link

coveralls commented Apr 13, 2021

Coverage Status

Coverage decreased (-0.4%) to 86.156% when pulling 523f547 on jwodder:packaging into 076c56f on duecredit:master.

@yarikoptic
Copy link
Member

I will drop python2 and resolve conflicts (via merge)

unable to resolve host travis-job-duecredit-duecredit-766907628
@jwodder
Copy link
Contributor Author

jwodder commented Apr 13, 2021

@yarikoptic If you're dropping Python 2 support, you should also get rid of the [bdist_wheel]universal setting in setup.cfg and set python_requires in setup.py.

@yarikoptic
Copy link
Member

thanks, done. Hopefully did it right ;-)

@yarikoptic
Copy link
Member

ok, CI is green. will not dive into coverage difference at this point. Thank you @jwodder -- next step is a release to prepare for #178

@yarikoptic yarikoptic merged commit 6c39499 into duecredit:master Apr 13, 2021
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.

None yet

3 participants