Skip to content

Add filters to stdlib.warnings.pyi #5889

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 5 commits into from
Aug 8, 2021
Merged

Conversation

hramezani
Copy link
Contributor

@bluetech suggested adding filters to warnings.pyi.

@github-actions

This comment has been minimized.

Copy link
Collaborator

@Akuli Akuli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are conflicts. Please run git pull https://github.com/python/typeshed and then resolve them.

@Akuli
Copy link
Collaborator

Akuli commented Aug 8, 2021

One more nit: Don't use git rebase or git commit --amend, so that you don't have to force-push. Force-pushing can erase previous code reviews.

@github-actions

This comment has been minimized.

2 similar comments
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@hramezani
Copy link
Contributor Author

Sorry for the conflict. I've addressed your comment based on your suggestion.
Thank you for the review.

Copy link
Collaborator

@Akuli Akuli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last tiny nit:

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2021

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

@Akuli Akuli merged commit 8b9d771 into python:master Aug 8, 2021
@hramezani hramezani deleted the warnings_filter branch August 8, 2021 20:19
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