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 beanshell to 2.0b6 #3294

Merged
merged 1 commit into from Oct 13, 2023
Merged

Update beanshell to 2.0b6 #3294

merged 1 commit into from Oct 13, 2023

Conversation

unp1
Copy link
Member

@unp1 unp1 commented Oct 4, 2023

No description provided.

@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #3294 (eb8b97d) into main (bf8cca5) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head eb8b97d differs from pull request most recent head 7f389a1. Consider uploading reports for the commit 7f389a1 to get more accurate results

@@             Coverage Diff              @@
##               main    #3294      +/-   ##
============================================
- Coverage     37.78%   37.75%   -0.03%     
- Complexity    16851    16864      +13     
============================================
  Files          2052     2052              
  Lines        125409   125697     +288     
  Branches      21220    21236      +16     
============================================
+ Hits          47386    47462      +76     
- Misses        72187    72398     +211     
- Partials       5836     5837       +1     

see 76 files with indirect coverage changes

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

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Thank you for your contribution.

The test artifacts are available on Artiweb.
The newest artifact is here.

@FliegendeWurst FliegendeWurst added the dependencies Pull requests that update a dependency file label Oct 5, 2023
@unp1 unp1 requested a review from Drodt October 13, 2023 09:38
@Drodt
Copy link
Member

Drodt commented Oct 13, 2023

I just got an error just now while running some more tests on this branch:

The tests testModelFieldTest_doubleX() and testModelFieldTest_f() in de.uka.ilkd.key.proof_references.testcase.analyst > TestClassAxiomAndInvariantProofReferencesAnalyst fail. Both, essentially, due to the same error:

AssertionFailedError: expected: <equals(test.ModelFieldTest::$f(heap,self),javaMulInt(Z(2(#)),int::select(heap,self,test.ModelFieldTest::$x)))> but was: <equals(test.ModelFieldTest::$f(heap,self),mul(Z(2(#)),int::select(heap,self,test.ModelFieldTest::$x)))>

The other result looks similar.

I suspect that this is not actually due to this PR. Thoughts, @unp1 ?

@wadoon wadoon added this pull request to the merge queue Oct 13, 2023
Merged via the queue into main with commit d48057e Oct 13, 2023
12 checks passed
@wadoon wadoon deleted the upgradeBeanshellVersion branch October 13, 2023 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants