-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rework maven code for faster calculation of path config #1661
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1661 +/- ##
=======================================
- Coverage 49% 49% -1%
+ Complexity 6312 6288 -24
=======================================
Files 664 665 +1
Lines 59626 59635 +9
Branches 8646 8647 +1
=======================================
- Hits 29490 29452 -38
- Misses 27924 27970 +46
- Partials 2212 2213 +1 ☔ View full report in Codecov by Sentry. |
On windows increases speed from 14s to 2s
…y, changing output streams, hoping it's not a race)
The idea would be to turn this PR into a general utility/service for Maven-related features for Rascal:
Davy's setup allows for a lean and fast application of maven features. |
Co-authored-by: Davy Landman <[email protected]>
…ependency injection in maven-embedder
…ting maven classpath to disk
a4d1565
to
718a6b3
Compare
718a6b3
to
2aab716
Compare
…call as all output is suppressed anyway
This needs effort to make this work together with our shaded jars.