Skip to content

Conversation

@mroeschke
Copy link
Member

I believe the intent of -W error was to catch DeprecationWarnings from numpy primarily. Currently builds are failing because of a warning from distutils which idk if we can do anything about.

@mroeschke mroeschke added the CI Continuous Integration label Apr 13, 2022
pattern: "not slow and not network and not single_cpu"
pandas_testing_mode: "deprecate"
test_args: "-W error"
test_args: "-W error::DeprecationWarning:numpy"
Copy link
Member

Choose a reason for hiding this comment

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

will this still catch FutureWarnings that we issue?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope, only the DeprecationWarnings from numpy. Do we want to fail the build on our FutureWarnings?

Copy link
Contributor

Choose a reason for hiding this comment

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

its possible we want to (but not this PR)

@jreback jreback added this to the 1.5 milestone Apr 13, 2022
@jreback
Copy link
Contributor

jreback commented Apr 14, 2022

backport?

@mroeschke mroeschke changed the title CI: Catch DeprecationWarning only from numpy on Numpy Dev build CI: Fail Numpy Dev build on DeprecationWarnings from numpy only Apr 14, 2022
@mroeschke
Copy link
Member Author

backport?

Yup probably a good idea

@jreback jreback modified the milestones: 1.5, 1.4.3 Apr 14, 2022
@jreback jreback merged commit dafa5dd into pandas-dev:main Apr 14, 2022
@jreback
Copy link
Contributor

jreback commented Apr 14, 2022

@meeseeksdev backport 1.4.x

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Apr 14, 2022
@mroeschke mroeschke deleted the ci/numpydev_warning branch April 14, 2022 01:11
lithomas1 pushed a commit that referenced this pull request Apr 14, 2022
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants