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

231 address first mypy errors #359

Closed
wants to merge 5 commits into from

Conversation

fhocutt
Copy link
Contributor

@fhocutt fhocutt commented May 14, 2018

Ignore errors on conditional imports (open mypy bug), explicitly pass None to upload.main(), put mypy back into CI.

Depends on #357.

Paired with Tathagata Dasgupta.

* Rename the pep8 tox environment to lint and add mypy
* Use basepython 3.6 since mypy requires >= 3.4
* Update twine/utils.py so flake8/python 3.6 passes

Paired with Tathagata Dasgupta.
Ignore errors on conditional imports (open mypy bug), explicitly pass
None to upload.main(), put mypy back into CI.
@jaraco
Copy link
Member

jaraco commented Jun 8, 2019

Thanks for the contribution, and sorry it was ignored for so long.

As I understand it, #357 was superseded by #380. Please rebase these changes on master if there's still any value to them.

@jaraco jaraco closed this Jun 8, 2019
@bhrutledge bhrutledge mentioned this pull request Jun 8, 2019
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