Skip to content

Make pedantic warnings opt-in in ci/build_X scripts#8218

Open
alliepiper wants to merge 2 commits intoNVIDIA:mainfrom
alliepiper:8217_script_options
Open

Make pedantic warnings opt-in in ci/build_X scripts#8218
alliepiper wants to merge 2 commits intoNVIDIA:mainfrom
alliepiper:8217_script_options

Conversation

@alliepiper
Copy link
Copy Markdown
Contributor

Add CCCL_ENABLE_PRAGMA_SYSTEM_HEADER CMake option and --pedantic flag to ci/build_common.sh. Local builds now default to warnings suppressed and werror off. CI (GITHUB_ACTIONS) auto-enables pedantic mode. Direct cmake preset invocations retain existing defaults (werror on, system header pragma off). Lit test plumbing propagates the setting to Python config.

Fixes #8217

Add CCCL_ENABLE_PRAGMA_SYSTEM_HEADER CMake option and --pedantic flag to
ci/build_common.sh. Local builds now default to warnings suppressed and
werror off. CI (GITHUB_ACTIONS) auto-enables pedantic mode. Direct cmake
preset invocations retain existing defaults (werror on, system header
pragma off). Lit test plumbing propagates the setting to Python config.

Fixes NVIDIA#8217
@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot bot commented Mar 30, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@alliepiper alliepiper requested a review from jrhemstad March 30, 2026 20:46
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Progress in CCCL Mar 30, 2026
@alliepiper
Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot bot commented Mar 30, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@alliepiper
Copy link
Copy Markdown
Contributor Author

/ok to test 2aef78b

@alliepiper alliepiper marked this pull request as ready for review March 30, 2026 23:22
@alliepiper alliepiper requested review from a team as code owners March 30, 2026 23:22
@cccl-authenticator-app cccl-authenticator-app bot moved this from In Progress to In Review in CCCL Mar 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🥳 CI Workflow Results

🟩 Finished in 2h 52m: Pass: 100%/445 | Total: 13d 15h | Max: 2h 41m | Hits: 82%/514771

See results here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

Default behavior of ci/build_X scripts should not surface warnings nor cause warnings as errors

3 participants