Skip to content

ENH: Add -Werror to CI (or at least show warnings) #55982

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

Closed
1 of 3 tasks
WillAyd opened this issue Nov 16, 2023 · 2 comments · Fixed by #56277
Closed
1 of 3 tasks

ENH: Add -Werror to CI (or at least show warnings) #55982

WillAyd opened this issue Nov 16, 2023 · 2 comments · Fixed by #56277
Labels
CI Continuous Integration Enhancement

Comments

@WillAyd
Copy link
Member

WillAyd commented Nov 16, 2023

Feature Type

  • Adding new functionality to pandas

  • Changing existing functionality in pandas

  • Removing existing functionality in pandas

Problem Description

Not sure if this was lost along the way but we originally did this in #38087

After #55151 was merged both @jbrockmendel and @rhshadrach had a lot of compiler warnings pop up. Both are on macOS, so I would have expected the macOS jobs to fail or at the very least still show compiler warnings, but am not seeing anything

@lithomas1 you may have already looked into this

Feature Description

Make sure compiler errors cause CI failures

Alternative Solutions

n/a

Additional Context

No response

@WillAyd WillAyd added Enhancement Needs Triage Issue that has not been reviewed by a pandas team member and removed Needs Triage Issue that has not been reviewed by a pandas team member labels Nov 16, 2023
@mroeschke mroeschke added the CI Continuous Integration label Nov 16, 2023
@mroeschke
Copy link
Member

Looks like -Werror is still in our setup.py, but not in our meson.build.

Also appears the debugging symbols (when requested) and MacOS specific Cython flags are missing too, but not too sure how necessary those are still to keep

@WillAyd
Copy link
Member Author

WillAyd commented Nov 16, 2023

Good callouts. The debugging symbols should be replaced with --config-settings=setup-args="-Dbuildtype=debug" during the pip install and meson-python should handle that correctly. Not sure about the macOS stuff but that is mostly for wheel building, not necessarily our CI

Haven't tested but I assume something like --config-settings=setup-args="--werror" should work for meson in CI

https://meson-python.readthedocs.io/en/latest/how-to-guides/meson-args.html

@WillAyd WillAyd mentioned this issue Dec 1, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants