Skip to content

Conversation

harinik
Copy link
Contributor

@harinik harinik commented Sep 29, 2025

Adds tests for #43164

@pytest.mark.parametrize("input", ["21-01-01", "01-01-21"])
@pytest.mark.parametrize("dayfirst", [True, False])
def test_parse_datetime_string_with_reso_dayfirst(dayfirst, input):
set_option("display.date_dayfirst", dayfirst)
Copy link
Member

Choose a reason for hiding this comment

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

Can you use the option_context context manager instead (and below)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated, PTAL.

@mroeschke mroeschke added the Testing pandas testing functions or related to the test suite label Sep 30, 2025
@harinik harinik requested a review from mroeschke October 3, 2025 21:21
@mroeschke mroeschke added this to the 3.0 milestone Oct 5, 2025
@mroeschke mroeschke merged commit 21f2ebe into pandas-dev:main Oct 5, 2025
42 checks passed
@mroeschke
Copy link
Member

Thanks @harinik

jzwick pushed a commit to jzwick/pandas that referenced this pull request Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: options date_dayfirst and date_yearfirst conflict
3 participants