Skip to content

Remove std scheme#2828

Draft
toinehartman wants to merge 11 commits into
mainfrom
feature/2817-remove-std-scheme/2819-pathconfig
Draft

Remove std scheme#2828
toinehartman wants to merge 11 commits into
mainfrom
feature/2817-remove-std-scheme/2819-pathconfig

Conversation

@toinehartman

@toinehartman toinehartman commented Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Closes #2817.

@toinehartman toinehartman self-assigned this Jul 16, 2026
@codecov

codecov Bot commented Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 47.05882% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 46%. Comparing base (4ce5e83) to head (f208de3).

Files with missing lines Patch % Lines
src/org/rascalmpl/library/util/PathConfig.java 51% 12 Missing and 1 partial ⚠️
...c/org/rascalmpl/shell/AbstractCommandlineTool.java 0% 2 Missing ⚠️
src/org/rascalmpl/shell/ShellEvaluatorFactory.java 50% 2 Missing ⚠️
.../org/rascalmpl/uri/StandardLibraryURIResolver.java 0% 1 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##              main   #2828   +/-   ##
=======================================
- Coverage       46%     46%   -1%     
+ Complexity    6724    6716    -8     
=======================================
  Files          842     841    -1     
  Lines        66796   66799    +3     
  Branches      9983    9987    +4     
=======================================
- Hits         30793   30776   -17     
- Misses       33622   33634   +12     
- Partials      2381    2389    +8

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

@toinehartman

toinehartman commented Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Integration test run: https://github.com/usethesource/rascal-core-big-tests/actions/runs/29577427795

toinehartman and others added 4 commits July 17, 2026 12:03
@toinehartman @rodinaarssen
Rename current runtime resolver.
fa40755 
Co-authored-by: Rodin Aarssen <rodin.aarssen@swat.engineering>
@toinehartman
Remove std:/// from Rascal path config."
3c038e5 
Part of #2819.
@toinehartman @rodinaarssen
Remove std:/// from Rascal-LSP path config.
78c3c58 
Part of #2819.

Co-authored-by: Rodin Aarssen <rodin.aarssen@swat.engineering>
@toinehartman @rodinaarssen
Remove std:/// from regular project's path config.
e337ef1 
Part of #2819.

Co-authored-by: Rodin Aarssen <rodin.aarssen@swat.engineering>
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from 47dfb34 to aac53b3 Compare July 17, 2026 10:12
@toinehartman
toinehartman changed the base branch from feature/2817-remove-std-scheme/final to main July 17, 2026 10:16
@toinehartman toinehartman changed the title Remove std scheme from path configs Remove std scheme Jul 17, 2026
This was referenced Jul 17, 2026
Closed
Closed
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from 5dbea58 to 34d136f Compare July 17, 2026 10:51
toinehartman and others added 4 commits July 17, 2026 12:59
@toinehartman @rodinaarssen
Replace std:/// on evaluator search path.
6d3b185 
Closes #2821.

Co-authored-by: Rodin Aarssen <rodin.aarssen@swat.engineering>
@toinehartman @rodinaarssen
Replace uses of std:/// in Rascal code.
30a9a8e 
Closes #2825.

Co-authored-by: Rodin Aarssen <rodin.aarssen@swat.engineering>
@toinehartman @rodinaarssen
Disable resolver for std:///.
ee58f72 
Closes #2822.

Co-authored-by: Rodin Aarssen <rodin.aarssen@swat.engineering>
@toinehartman
Package stdlib locations as mvn://
64173b4 
This removes `std://` locations in the published Rascal JARs, in favor
of `mvn://` pointing to this specific released version. Closes #2820.

Closes #2820.
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from 34d136f to 64173b4 Compare July 17, 2026 11:00
@sonarqubecloud

sonarqubecloud Bot commented Jul 17, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

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

Reviewers

@DavyLandman DavyLandman Awaiting requested review from DavyLandman DavyLandman will be requested when the pull request is marked ready for review DavyLandman is a code owner

Assignees

@toinehartman toinehartman

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove the std scheme

2 participants

@toinehartman @rodinaarssen