Skip to content

Remove std scheme - #2828

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

Remove std scheme#2828
toinehartman wants to merge 34 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 50.72464% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 45%. Comparing base (910b39b) to head (9e02238).

Files with missing lines Patch % Lines
src/org/rascalmpl/library/util/PathConfig.java 54% 11 Missing and 4 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       45%     45%           
  Complexity    6722    6722           
=======================================
  Files          844     843    -1     
  Lines        66994   66980   -14     
  Branches     10010   10008    -2     
=======================================
+ Hits         30791   30793    +2     
+ Misses       33816   33799   -17     
- Partials      2387    2388    +1

☔ 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
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from b4aee93 to 7f972f8 Compare August 6, 2026 20:35
@toinehartman
toinehartman force-pushed the feature/2817-remove-std-scheme/2819-pathconfig branch from 29d9b51 to 9e02238 Compare August 7, 2026 08:10
@sonarqubecloud

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