Skip to content

Conversation

@srittau
Copy link
Collaborator

@srittau srittau commented Mar 22, 2022

No description provided.

@srittau
Copy link
Collaborator Author

srittau commented Mar 22, 2022

Oh well. 👀

@srittau srittau closed this Mar 22, 2022
@srittau srittau reopened this Mar 22, 2022
@srittau
Copy link
Collaborator Author

srittau commented Mar 22, 2022

Weirdly enough, I can't reproduce the CI failures when running ./tests/mypy_test.py locally.

@AlexWaygood
Copy link
Member

Weirdly enough, I can't reproduce the CI failures when running ./tests/mypy_test.py locally.

There are also still some pytype errors.

This reverts commit 36e34a6.
@JukkaL
Copy link
Contributor

JukkaL commented Mar 23, 2022

I'm investigating the mypy errors.

@srittau srittau marked this pull request as draft March 23, 2022 14:40
@AlexWaygood AlexWaygood reopened this Mar 24, 2022
@github-actions

This comment has been minimized.

@srittau
Copy link
Collaborator Author

srittau commented Mar 24, 2022

Cool, just pytype left, and that has landed a fix as well (unreleased). Edit: google/pytype#1159

@github-actions
Copy link
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

@srittau srittau marked this pull request as ready for review March 30, 2022 09:05
@srittau srittau merged commit 9687d53 into python:master Mar 30, 2022
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.

3 participants