Skip to content
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

Deprecate passing args as positional in dropna #41504

Merged

Conversation

MarcoGorelli
Copy link
Member

@MarcoGorelli MarcoGorelli added the Deprecate Functionality to remove in pandas label May 16, 2021
@MarcoGorelli MarcoGorelli force-pushed the deprecate-nonkeyword-args-dropna branch from 31f7cbc to 2660810 Compare May 16, 2021 12:15
@@ -231,3 +231,13 @@ def test_dropna_with_duplicate_columns(self):

result = df.dropna(subset=["A", "C"], how="all")
tm.assert_frame_equal(result, expected)

def test_dropna_pos_args_deprecation(self):
Copy link
Member

Choose a reason for hiding this comment

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

could probably share this test using frame_or_series fixture?

r"argument 'self' will be keyword-only"
)
with tm.assert_produces_warning(FutureWarning, match=msg):
df.dropna(1)
Copy link
Member

Choose a reason for hiding this comment

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

Similar as in other PRs, can you still assert the result?

@jorisvandenbossche jorisvandenbossche changed the title deprecate passing args as positional in dropna Deprecate passing args as positional in dropna May 17, 2021
@MarcoGorelli MarcoGorelli marked this pull request as draft May 18, 2021 08:14
@MarcoGorelli MarcoGorelli marked this pull request as ready for review May 19, 2021 19:56
@MarcoGorelli MarcoGorelli force-pushed the deprecate-nonkeyword-args-dropna branch from 8c0a0d7 to d4ad702 Compare May 19, 2021 19:58
@simonjayhawkins simonjayhawkins added this to the 1.3 milestone May 24, 2021
@jreback jreback merged commit d662e97 into pandas-dev:master May 26, 2021
@MarcoGorelli MarcoGorelli deleted the deprecate-nonkeyword-args-dropna branch May 26, 2021 20:52
TLouf pushed a commit to TLouf/pandas that referenced this pull request Jun 1, 2021
JulianWgs pushed a commit to JulianWgs/pandas that referenced this pull request Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants