Skip to content

More Bundler improvements - #12400

Merged
sschuberth merged 3 commits into
mainfrom
more-bundler-imps
Sep 4, 2026
Merged

More Bundler improvements#12400
sschuberth merged 3 commits into
mainfrom
more-bundler-imps

Conversation

@sschuberth

Copy link
Copy Markdown
Member

Please have a look at the individual commit messages for the details.

@sschuberth
sschuberth requested a review from a team as a code owner September 4, 2026 06:35
@sschuberth
sschuberth enabled auto-merge (rebase) September 4, 2026 06:40
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.13%. Comparing base (b952ad1) to head (9a91cad).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #12400   +/-   ##
=========================================
  Coverage     59.13%   59.13%           
  Complexity     1878     1878           
=========================================
  Files           365      365           
  Lines         13810    13810           
  Branches       1441     1441           
=========================================
  Hits           8166     8166           
  Misses         5113     5113           
  Partials        531      531           
Flag Coverage Δ
funTest-external-tools 15.85% <ø> (ø)
test-windows-2025 42.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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 plugins/package-managers/bundler/src/main/kotlin/Bundler.kt Outdated
Reduce a bit of code duplication.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.io>
This is a preparation for an upcoming change.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.io>
Instead of implying the Gemfile to use by the working directory, pass it
explicitly via the `BUNDLE_GEMFILE` environment variable.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.io>
@sschuberth
sschuberth merged commit 57e36a7 into main Sep 4, 2026
33 checks passed
@sschuberth
sschuberth deleted the more-bundler-imps branch September 4, 2026 15:03
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