Skip to content

Add a --warn-error flag - #1718

Merged
Alasdair merged 1 commit into
sail2from
warn_error
Jul 30, 2026
Merged

Add a --warn-error flag#1718
Alasdair merged 1 commit into
sail2from
warn_error

Conversation

@Alasdair

Copy link
Copy Markdown
Collaborator

This flag promotes warnings to errors. It only works in conjunction with the --required-version flag. Each warning now and going forwards, has a specified version where it was introduced (all current warnings are marked as 0.20.2). Any warning with an introduced version less than or equal to the required version will become an error. This means that new warnings can be introduced without breaking downstream CI builds that want to use the flag.

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

Test Results

   17 files     38 suites   0s ⏱️
1 056 tests 1 053 ✅  3 💤 0 ❌
5 400 runs  5 355 ✅ 45 💤 0 ❌

Results for commit ad07af7.

♻️ This comment has been updated with latest results.

@Alasdair
Alasdair force-pushed the warn_error branch 2 times, most recently from 9eb6337 to ca7dabf Compare July 30, 2026 15:55
This flag promotes warnings to errors. It only works in conjunction with the
`--required-version` flag. Each warning now--and going forwards, has a specified
version where it was introduced (all current warnings are marked as 0.20.2). Any
warning with an introduced version less than or equal to the required version
will become an error. This means that new warnings can be introduced without
breaking downstream CI builds that want to use the flag.
@Alasdair
Alasdair merged commit fc6abe2 into sail2 Jul 30, 2026
18 checks passed
@Alasdair
Alasdair deleted the warn_error branch July 30, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant