Skip to content

Proof Management: Ignore internal contracts when calculating dependency state (fixes #3745)#3748

Merged
WolframPfeifer merged 3 commits intomainfrom
pfeifer/3745pmIgnoreInternal
Feb 27, 2026
Merged

Proof Management: Ignore internal contracts when calculating dependency state (fixes #3745)#3748
WolframPfeifer merged 3 commits intomainfrom
pfeifer/3745pmIgnoreInternal

Conversation

@WolframPfeifer
Copy link
Member

This PR fixes an issue with calculation of dependencies: Dependencies to contracts that are not in user-provided sources (e.g., in bootclasspath) are not reported as unproven dependencies any more. However, they are still shown in the "Dependencies" tab in the report, and are still printed on the console with INFO log level.

Related Issue

This pull request resolves #3745.

Type of pull request

  • Bug fix (non-breaking change which fixes an issue)
  • There are changes to the (Java) code

Ensuring quality

Additional information and contact(s)

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

@WolframPfeifer WolframPfeifer added this to the v3.0.0 milestone Feb 18, 2026
@WolframPfeifer WolframPfeifer self-assigned this Feb 18, 2026
@WolframPfeifer WolframPfeifer added 🐞 Bug Java Pull requests that update Java code keyext.proofmanagement Module: keyext.proofmanagement labels Feb 18, 2026
Copy link
Member

@FliegendeWurst FliegendeWurst left a comment

Choose a reason for hiding this comment

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

This fix works on my bigger example as well!

@WolframPfeifer WolframPfeifer added this pull request to the merge queue Feb 27, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 27, 2026
@WolframPfeifer WolframPfeifer added this pull request to the merge queue Feb 27, 2026
@WolframPfeifer
Copy link
Member Author

I don't understand why the formatting task fails when trying to merge. On the updated branch (before a fast-forward merge to main), the formatting task was successful.

It seems to be a configuration problem, not an actual formatting issue.

Merged via the queue into main with commit 25c50a7 Feb 27, 2026
36 checks passed
@WolframPfeifer WolframPfeifer deleted the pfeifer/3745pmIgnoreInternal branch February 27, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Java Pull requests that update Java code keyext.proofmanagement Module: keyext.proofmanagement 🐞 Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proof management report claims proofs for JavaRedux methods are missing

3 participants