android / android/ndk

Force warnings on in the test runner

Open
#388 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.