python / python/cpython

Consider running `Hypothesis` tests with `-We`

Aperta
#136,409 0 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@sobolevn ci sta già lavorando.

Dal 8/7/2025.

infra tests
Lingua principale
Python
Stelle
77.2k
Fork
35.9k
Metriche di merge delle PR
Metriche PR in attesa

Descrizione

Currently we never run tests in CI with -We.

» ag '\-W(e| error)' .github
.github/workflows/build.yml
79:        run: autoreconf -ivf -Werror
                        

This might be a problem for several reasons:

  • Warnings might be raised and unseen for quite some time
  • -We can break something that it should not break, we never check for it

This is especially true for Hypothesis runs, which has its own warnings, sometimes not very expected ones, like in https://github.com/python/cpython/pull/136384

My proposal: let's try to research if it is possible and if it is a good idea. And maybe eventually run Hypothesis tests with -We in our CI.

Please, share your ideas / concerns about this change.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.