Skip to content

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

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

WolframPfeifer merged 3 commits into
mainfrom
pfeifer/3745pmIgnoreInternal

Conversation

@WolframPfeifer

Copy link
Copy Markdown
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

@FliegendeWurst FliegendeWurst left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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