Skip to content

gh-133117: Enable stricter mypy checks for tomllib #133206

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
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sobolevn
Copy link
Member

@sobolevn sobolevn commented Apr 30, 2025

@AlexWaygood
Copy link
Member

I support this but I'm not sure if it makes sense to merge it prior to hukkin/tomli#257? Might it not cause problems if the two codebases get out of sync?

I'll defer to @hukkin / @hauntsaninja / @encukou as the tomllib maintainers :-)

@AlexWaygood AlexWaygood removed their request for review April 30, 2025 13:39
@sobolevn
Copy link
Member Author

sobolevn commented May 1, 2025

hukkin/tomli#257 was merged. The only difference with this PR is that here we still use --incremental, I think it is more important for us, since the scale of the project and the fact that we don't have any extra configs.

cc @hukkin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants