-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: check_freq=False #34050
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
CI: check_freq=False #34050
Conversation
@@ -1140,6 +1141,8 @@ def assert_series_equal( | |||
Whether to compare internal Categorical exactly. | |||
check_category_order : bool, default True | |||
Whether to compare category order of internal Categoricals. | |||
check_freq : bool, default True |
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.
move to below versionadded:: 1.0.2 for check_category_order and add versionadded:: 1.1.0
check_freq : bool, default True | ||
Whether to check the `freq` attribute on a DatetimeIndex or TimedeltaIndex. |
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.
add versionadded:: 1.1.0
looks ok, comments by @simonjayhawkins |
comments addressed |
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.
lgtm merge into green
we should maybe make the default False for backwards compat here (pls open an issue)
green |
k thanks. pls open an issue to change this to |
Jepp, this broke some code for me. Will open the mentioned issue. |
Aimed at a test that has been periodically failing in the CI for a week or so.