-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
Issue 33428 fix #33583
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
Issue 33428 fix #33583
Conversation
This reverts commit 6a6fde1.
Hello @Dxin-code! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
Thanks @Dxin-code for the PR. #33428 is already fixed? issue is closed. |
@@ -219,7 +219,7 @@ def test_frame_equal_unicode(df1, df2, msg, by_blocks_fixture, obj_fixture): | |||
# | |||
# Test ensures that `tm.assert_frame_equals` raises the right exception | |||
# when comparing DataFrames containing differing unicode objects. | |||
msg = msg.format(obj=obj_fixture) | |||
msg = f"{obj_fixture}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please revert this change in this PR. this change is also included in #33582?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The other edit fails the linter because the line is too long. i think close this and focus on the other PRs
Thanks for the PR but looks like this was already solved in #33099 |
Got it! Thanks for the reminder! |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff