Skip to content

fix: perform severity overrides before counting errors#13589

Merged
Kha merged 1 commit intoleanprover:masterfrom
eric-wieser:patch-75
May 5, 2026
Merged

fix: perform severity overrides before counting errors#13589
Kha merged 1 commit intoleanprover:masterfrom
eric-wieser:patch-75

Conversation

@eric-wieser
Copy link
Copy Markdown
Contributor

@eric-wieser eric-wieser commented Apr 30, 2026

This PR ensures that the lean --error=tag flag actually sets a non-zero exit code on promoted errors.

Closes #13588, fixing the feature introduced in #6362.

This PR ensures that the `lean --error=tag` flag actually sets a non-zero exit code on promoted errors.

Fixes leanprover#13588
@eric-wieser eric-wieser requested a review from Kha as a code owner April 30, 2026 17:07
@eric-wieser
Copy link
Copy Markdown
Contributor Author

changelog-compiler

@github-actions github-actions Bot added changelog-compiler Compiler, runtime, and FFI toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN labels Apr 30, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 427e3bcdbca175a69806cbd6d76ecd130379bfba --onto 0a6c31520b2ebaa6e71227454b46ebfb4986ae7b. You can force Mathlib CI using the force-mathlib-ci label. (2026-04-30 18:00:57)

@leanprover-bot
Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 427e3bcdbca175a69806cbd6d76ecd130379bfba --onto 3fc99eef102549c743c6c63547983319bfae6f01. You can force reference manual CI using the force-manual-ci label. (2026-04-30 18:00:58)

@Kha Kha added changelog-other and removed changelog-compiler Compiler, runtime, and FFI labels May 5, 2026
@Kha Kha enabled auto-merge May 5, 2026 09:31
@Kha Kha added this pull request to the merge queue May 5, 2026
Merged via the queue into leanprover:master with commit e4c0a5a May 5, 2026
32 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-other toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lean --error does not result in a non-zero exit code

3 participants