Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

repair the axiom generation for adts defined in JavaDL #3556

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

mattulbrich
Copy link
Member

Related Issue

This pull request addresses #3427, but does not solve it.

Intended Change

The axiom taclet created for adt was not applicable. It is applicable now.

Plan

  • remove the "notFreeIn" clause from the generated taclet
  • remove the find clause from the taclet.

Type of pull request

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

Ensuring quality

  • I have tested the feature as follows: Looked at the translation of a few examples

It is a small change in the creation of a taclet.

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

@mattulbrich mattulbrich requested a review from wadoon February 19, 2025 15:26
@mattulbrich
Copy link
Member Author

Addressed during the 3rd HacKeYthon.

Copy link

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.33%. Comparing base (a0c4f81) to head (46ef350).
Report is 13 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3556      +/-   ##
============================================
- Coverage     38.33%   38.33%   -0.01%     
+ Complexity    17258    17257       -1     
============================================
  Files          2111     2111              
  Lines        127632   127630       -2     
  Branches      21461    21461              
============================================
- Hits          48926    48923       -3     
  Misses        72695    72695              
- Partials       6011     6012       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@wadoon wadoon added this pull request to the merge queue Feb 20, 2025
Merged via the queue into KeYProject:main with commit 824af63 Feb 20, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants