Consider running `Hypothesis` tests with `-We`
Đang mở
@sobolevn đang làm issue này rồi.
Từ ngày 8/7/2025.
infra
tests
- Ngôn ngữ chính
- Python
- Star
- 77.2k
- Fork
- 35.9k
- Chỉ số merge pull request
- Chỉ số pull request đang chờ
Mô tả
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
-Wecan 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.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Đánh giá
Issue này chưa được đánh giá.