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

DEPR: allowing subclass-specific keywords in pd.Index.__new__ #38597

Merged
merged 10 commits into from
Dec 22, 2020

Conversation

jbrockmendel
Copy link
Member

@@ -506,7 +506,9 @@ def test_insert_index_period(self, insert, coerced_val, coerced_dtype):
else:
msg = r"Unexpected keyword arguments {'freq'}"
with pytest.raises(TypeError, match=msg):
pd.Index(data, freq="M")
with tm.assert_produces_warning(FutureWarning):
# passing keywords ot pd.Index
Copy link
Member

Choose a reason for hiding this comment

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

typo in comment

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

looks good. a testing request and some small comments. ping on green.

pandas/core/resample.py Show resolved Hide resolved
doc/source/whatsnew/v1.3.0.rst Outdated Show resolved Hide resolved
@jreback jreback added Constructors Series/DataFrame/Index/pd.array Constructors Deprecate Functionality to remove in pandas labels Dec 21, 2020
@jreback jreback added this to the 1.3 milestone Dec 21, 2020
pandas/core/resample.py Outdated Show resolved Hide resolved
@jreback
Copy link
Contributor

jreback commented Dec 22, 2020

can you merge master once more. i realized that the period keyword removal was merged before this

@jreback
Copy link
Contributor

jreback commented Dec 22, 2020

somethings still failing in precommit

@jreback jreback merged commit 1bac7ac into pandas-dev:master Dec 22, 2020
@jreback
Copy link
Contributor

jreback commented Dec 22, 2020

nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Constructors Series/DataFrame/Index/pd.array Constructors Deprecate Functionality to remove in pandas
Projects
None yet
4 participants