Skip to content

Remove std scheme - #2828

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

Remove std scheme#2828
toinehartman wants to merge 26 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 #2819.
Closes #2821.
Closes #2820.
Closes #2825.
Closes #2822.
Closes #2830.

@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 52.23881% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 46%. Comparing base (4ce5e83) to head (ba81e03).

Files with missing lines Patch % Lines
src/org/rascalmpl/library/util/PathConfig.java 58% 11 Missing and 2 partials ⚠️
...ascalmpl/repl/completers/RascalQualifiedNames.java 0% 10 Missing ⚠️
...rascalmpl/uri/file/MavenRepositoryURIResolver.java 75% 1 Missing and 2 partials ⚠️
...c/org/rascalmpl/shell/AbstractCommandlineTool.java 0% 2 Missing ⚠️
src/org/rascalmpl/shell/ShellEvaluatorFactory.java 50% 2 Missing ⚠️
...nterpreter/load/SourceLocationListContributor.java 0% 1 Missing ⚠️
...mpl/uri/remote/jsonrpc/SourceLocationResponse.java 0% 1 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##              main   #2828   +/-   ##
=======================================
- Coverage       46%     46%   -1%     
+ Complexity    6724    6717    -7     
=======================================
  Files          842     841    -1     
  Lines        66796   66782   -14     
  Branches      9983    9981    -2     
=======================================
- Hits         30793   30779   -14     
+ Misses       33622   33614    -8     
- 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
@toinehartman toinehartman mentioned this pull request Jul 20, 2026
Draft
@toinehartman toinehartman mentioned this pull request Jul 20, 2026
Open
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from 90f4730 to 3a90d49 Compare July 28, 2026 16:13
@toinehartman @rodinaarssen
Expose Maven parsing and resolution for use in Rascal LSP.
6d18180 
Co-authored-by: rodinaarssen <rodin.aarssen@swat.engineering>
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from 3a90d49 to 6d18180 Compare July 28, 2026 16:17
@sonarqubecloud

sonarqubecloud Bot commented Jul 30, 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

@rodinaarssen rodinaarssen

Labels

None yet

Projects

None yet

Milestone

No milestone

2 participants

@toinehartman @rodinaarssen