Skip to content

Integration with mypy_primer? #122

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

Open
hauntsaninja opened this issue Apr 18, 2025 · 4 comments
Open

Integration with mypy_primer? #122

hauntsaninja opened this issue Apr 18, 2025 · 4 comments

Comments

@hauntsaninja
Copy link

Hello! Let me know if I can make https://github.com/hauntsaninja/mypy_primer useful for pyrefly

@MaggieMoss
Copy link
Contributor

Hi! We would love to integrate into mypy_primer. What would be the best way to go about it? We can put up a PR and collaborate that way?

@hauntsaninja
Copy link
Author

Yup! I also added a few lines on how to get set up here: https://github.com/hauntsaninja/mypy_primer#adding-support-for-a-new-type-checker

@yangdanny97
Copy link
Contributor

yangdanny97 commented Apr 19, 2025

Last week I got pyrefly working with mypy_primer in a fork at https://github.com/yangdanny97/mypy_primer, the basic behavior should be there but the per-project commands/configs need some changes.

If I opened a PR and we merged it now, would it affect the CI jobs for everyone that uses the primer? If so, we may want to wait till Pyrefly is more stable to reduce churn.

We're also running a version of mypy_primer in our internal CI, but it's an entirely new script that only reuses the per-project configs from the original mypy_primer. The main differences are that it uses mercurial instead of git, and it doesn't do a diff of before/after - it just runs once and logs the errors to a table for us to track the errors over time.

We'd like to run the regular version of mypy_primer in CI at some point though.

@hauntsaninja
Copy link
Author

Sounds good! I think some of your diff would no longer be needed given some of my latest changes. See also hauntsaninja/mypy_primer#155

The pyrefly branches are self-contained, so it won't really affect other projects' jobs if it were to be merged, so happy to have it upstreamed now

Also happy to grant some of y'all commit access, in case that makes anything easier

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

No branches or pull requests

3 participants