Force warnings on in the test runner
Open
- Dominant language
- No language data
- Stars
- 2.3k
- Forks
- 310
- PR merge metrics
- No merged PRs in 30d
Description
We should force warnings on in the test runner (at least `-Wall -Wextra -Werror`, maybe even `-Weverything` with Clang). We want to catch any issues in our code so we don't prevent users from using these flags.
Contributor guide
Research direction
Start by locating the test runner and reviewing how compiler flags are configured for its builds. Check the runner with GCC and Clang using -Wall, -Wextra, and -Werror, then assess whether -Weverything is viable. Done means the runner enforces the agreed warning flags and the existing tests still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- build-system, testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100