Skip to content

Conversation

@wadoon
Copy link
Member

@wadoon wadoon commented Nov 8, 2025

Related Issue

Assertions are not enabled when KeY is run by Gradle. Hence, assertions are not checked during the quality assurance of releases.

Intended Change

  1. -ea set as JVM argument for :key.ui:run
  2. Report whether assert is enabled on console.

Type of pull request

  • Bug fix (non-breaking change which fixes an issue)

Ensuring quality

  • I have tested the feature as follows: Manually inspection.

@wadoon wadoon self-assigned this Nov 8, 2025
@wadoon wadoon added the 🐞 Bug label Nov 8, 2025
@wadoon wadoon requested a review from unp1 November 8, 2025 21:19
@wadoon wadoon added this to the v2.12.4 milestone Nov 8, 2025
Copy link
Member

@unp1 unp1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot. Only thing left is formatting

@wadoon wadoon enabled auto-merge November 13, 2025 19:36
@wadoon wadoon added this pull request to the merge queue Nov 13, 2025
Merged via the queue into main with commit 8c09511 Nov 13, 2025
35 checks passed
@wadoon wadoon deleted the weigl/ea branch November 13, 2025 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants