Skip to content

TST,WARN: read_csv raises warning at wrong stacklevel, but _assert_raised_with_correct_stacklevel doesn't fail #47387

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

Conversation

MarcoGorelli
Copy link
Member

@MarcoGorelli MarcoGorelli added the Warnings Warnings that appear or should be added to pandas label Jun 16, 2022
@mroeschke mroeschke added this to the 1.5 milestone Jun 23, 2022
Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

Do you think this needs a whatsnew for 1.5?

@MarcoGorelli
Copy link
Member Author

MarcoGorelli commented Jun 24, 2022

Do you think this needs a whatsnew for 1.5?

yes, probably, thanks

should this be backported too, as it affects legibility of warnings to do with an upcoming deprecation?

@mroeschke
Copy link
Member

Do you think this needs a whatsnew for 1.5?

yes, probably, thanks

should this be backported too, as it affects legibility of warnings to do with an upcoming deprecation?

Yeah I think this would be okay to backport cc @simonjayhawkins

@mroeschke mroeschke modified the milestones: 1.5, 1.4.4 Jun 24, 2022
@simonjayhawkins
Copy link
Member

Yeah I think this would be okay to backport cc @simonjayhawkins

sure. there's a bug fix section in the 1.4.4 release notes.

@MarcoGorelli MarcoGorelli merged commit 109be48 into pandas-dev:main Jun 26, 2022
@lumberbot-app

This comment was marked as resolved.

MarcoGorelli added a commit to MarcoGorelli/pandas that referenced this pull request Jun 26, 2022
…ong stacklevel, but _assert_raised_with_correct_stacklevel doesn't fail
simonjayhawkins pushed a commit that referenced this pull request Jul 2, 2022
…g at wrong stacklevel, but _assert_raised_with_correct_stacklevel doesn't fail )" (#47513)
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
…ised_with_correct_stacklevel doesn't fail (pandas-dev#47387)

* fix stacklevel of read_csv warning

* :doc: add whatsnewnote

* move whatsnew entry to v1.4.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Warnings Warnings that appear or should be added to pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TST,WARN: read_csv raises warning at wrong stacklevel, but _assert_raised_with_correct_stacklevel doesn't fail
3 participants