Skip to content

Preparing anno to kw param quickfix for inclusion in VScode#2793

Merged
DavyLandman merged 4 commits into
mainfrom
feat/anno-quickfix
Jun 5, 2026
Merged

Preparing anno to kw param quickfix for inclusion in VScode#2793
DavyLandman merged 4 commits into
mainfrom
feat/anno-quickfix

Conversation

@jurgenvinju

@jurgenvinju jurgenvinju commented Jun 3, 2026

Copy link
Copy Markdown
Member
  • prepare the annotation to kw field rewriter for inclusion in the IDE by calling treeDiff
  • this only calls the existing rewriting visit without changing anything about it. It has been applied on the standard library, the checker, the compiler, the tutor and all libraries we publish.
  • this is the first real application of treeDiff so ample testing required, but we can do that in the VScode extension because the integration of the edits with the commands is also interesting to test a bit deeper.
  • this uses a progress bar to loop over all the rascal source files in a project. interesting to see how it looks in VScode

@codecov

codecov Bot commented Jun 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46%. Comparing base (4c12581) to head (8b23730).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##              main   #2793   +/-   ##
=======================================
  Coverage       46%     46%           
- Complexity    6721    6727    +6     
=======================================
  Files          839     839           
  Lines        66766   66766           
  Branches      9983    9983           
=======================================
+ Hits         30780   30782    +2     
- Misses       33600   33602    +2     
+ Partials      2386    2382    -4     

☔ 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.

Comment thread src/org/rascalmpl/library/lang/rascal/upgrade/UpgradeBase.rsc Outdated
@sonarqubecloud

sonarqubecloud Bot commented Jun 5, 2026

Copy link
Copy Markdown

@DavyLandman DavyLandman merged commit 4e22b76 into main Jun 5, 2026
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants