Skip to content

Add filters to stdlib.warnings.pyi - #5889

Merged
Akuli merged 5 commits into
python:masterfrom
hramezani:warnings_filter
Aug 8, 2021
Merged

Akuli merged 5 commits into
python:masterfrom
hramezani:warnings_filter

Conversation

@hramezani

Copy link
Copy Markdown
Contributor

@bluetech suggested adding filters to warnings.pyi.

@github-actions

This comment has been minimized.

Comment thread stdlib/warnings.pyi Outdated

@Akuli Akuli left a comment

Copy link
Copy Markdown
Collaborator

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.

Comment thread stdlib/warnings.pyi Outdated
Comment thread stdlib/_warnings.pyi Outdated
@Akuli

Akuli commented Aug 8, 2021

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Contributor Author

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

@Akuli Akuli left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Last tiny nit:

Comment thread stdlib/warnings.pyi Outdated
@github-actions

github-actions Bot commented Aug 8, 2021

Copy link
Copy Markdown
Contributor

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