Skip to content

mypy_primer: speed up, clean up #4730

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 2 commits into from
Oct 30, 2020
Merged

mypy_primer: speed up, clean up #4730

merged 2 commits into from
Oct 30, 2020

Conversation

hauntsaninja
Copy link
Collaborator

@hauntsaninja hauntsaninja commented Oct 29, 2020

Use a version pip recognises so that we can install mypyc compiled mypy
from PyPI

We use tee and no longer try to delete the file based on exit code,
since we use data.trim now.
mypy_primer started failing silently because of changes in aiohttp, so
now we'll fail if the exit codes are not 0 or 1.
Note mypy_primer has --project-date to get around the problem of
breaking changes in projects, but I want to try and keep mypy_primer up
to date / get a feel for how bad this problem actually is.

hauntsaninja added 2 commits October 29, 2020 16:03
We use tee and no longer try to delete the file based on exit code,
since we use data.trim now.
mypy_primer started failing silently because of changes in aiohttp, so
now we'll fail if the exit codes are not 0 or 1.
Note mypy_primer has --project-date to get around the problem of
breaking changes in projects, but I want to try and keep mypy_primer up
to date / get a feel for how bad this problem actually is.
Use a version pip recognises so that we can install mypyc compiled mypy
from PyPI
@hauntsaninja hauntsaninja changed the title mypy_primer: clean up command mypy_primer: speedup, clean up command Oct 30, 2020
@hauntsaninja hauntsaninja changed the title mypy_primer: speedup, clean up command mypy_primer: speed up, clean up Oct 30, 2020
@srittau srittau merged commit c58a93b into python:master Oct 30, 2020
@hauntsaninja hauntsaninja deleted the prim branch October 30, 2020 16:40
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