Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.7k
Open
Description
Currently we never run tests in CI with-We.
» ag '\-W(e| error)' .github.github/workflows/build.yml79: run: autoreconf -ivf -WerrorThis might be a problem for several reasons:
- Warnings might be raised and unseen for quite some time
-Wecan break something that it should not break, we never check for it
This is especially true forHypothesis runs, which has its own warnings, sometimes not very expected ones, like in#136384
My proposal: let's try to research if it is possible and if it is a good idea. And maybe eventually runHypothesis tests with-We in our CI.
Please, share your ideas / concerns about this change.