Skip to content

added syntax for improved any and all, next to the other comprehensions#2250

Draft
jurgenvinju wants to merge 2 commits into
mainfrom
improved-all-any
Draft

added syntax for improved any and all, next to the other comprehensions#2250
jurgenvinju wants to merge 2 commits into
mainfrom
improved-all-any

Conversation

@jurgenvinju

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented May 7, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 46%. Comparing base (81044e6) to head (5100fe0).
⚠️ Report is 24361 commits behind head on main.

Files with missing lines Patch % Lines
...rc/org/rascalmpl/semantics/dynamic/Expression.java 88% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##              main   #2250     +/-   ##
=========================================
- Coverage       47%     46%     -2%     
- Complexity    6339    6719    +380     
=========================================
  Files          764     839     +75     
  Lines        63177   66765   +3588     
  Branches      9431    9983    +552     
=========================================
+ Hits         29938   30770    +832     
- Misses       31018   33608   +2590     
- Partials      2221    2387    +166     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DavyLandman

Copy link
Copy Markdown
Member

Let's not merge this? Since it breaks concrete syntax in rascal compiler?

@DavyLandman DavyLandman force-pushed the improved-all-any branch 2 times, most recently from a1319e1 to 5100fe0 Compare June 26, 2026 12:02
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants