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

Update QuintIR for new parameter representation #2473

Merged
merged 2 commits into from
Mar 9, 2023
Merged

Conversation

shonfeder
Copy link
Contributor

@shonfeder shonfeder commented Mar 9, 2023

The apalache side fix corresponding to
informalsystems/quint#689


  • Tests added for any new code
  • Ran make fmt-fix (or had formatting run automatically on all files edited)
  • Documentation added for any new functionality
  • Entries added to ./unreleased/ for any new functionality

@shonfeder shonfeder requested a review from Kukovec as a code owner March 9, 2023 17:54
@codecov-commenter
Copy link

Codecov Report

Merging #2473 (048b0bf) into main (c6078f6) will decrease coverage by 0.01%.
The diff coverage is 75.00%.

❗ Current head 048b0bf differs from pull request most recent head 6b375a1. Consider uploading reports for the commit 6b375a1 to get more accurate results

@@            Coverage Diff             @@
##             main    #2473      +/-   ##
==========================================
- Coverage   78.36%   78.36%   -0.01%     
==========================================
  Files         441      441              
  Lines       15408    15408              
  Branches     2467     2453      -14     
==========================================
- Hits        12075    12074       -1     
- Misses       3333     3334       +1     
Impacted Files Coverage Δ
...n/scala/at/forsyte/apalache/io/quint/QuintIR.scala 100.00% <ø> (ø)
...ain/scala/at/forsyte/apalache/io/quint/Quint.scala 82.42% <75.00%> (ø)
...a/at/forsyte/apalache/tla/lir/TlaLevelFinder.scala 93.75% <0.00%> (-2.09%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Collaborator

@bugarela bugarela left a comment

Choose a reason for hiding this comment

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

Thanks for adapting this so quick 💖

@shonfeder shonfeder merged commit 3c59c6f into main Mar 9, 2023
@shonfeder shonfeder deleted the update-quint-ir branch March 9, 2023 18:44
@apalache-bot apalache-bot mentioned this pull request Mar 10, 2023
@shonfeder shonfeder self-assigned this Mar 31, 2023
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