-
Notifications
You must be signed in to change notification settings - Fork 990
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
Bundler 2 [pre-release]: Detect bundler version to use when bundler 2 is available #3350
Conversation
bundler/spec/dependabot/bundler/update_checker/conflicting_dependency_resolver_spec.rb
Outdated
Show resolved
Hide resolved
68dac8d
to
fd3d377
Compare
Odd, I didn't expect |
a4f321d
to
7cd7e2a
Compare
The only things failing on this PR now should be a handful of tests in the bundler2 build which aren't using project-based fixtures. Since that will make them easy to find, my plan is a follow up commit which fixes this we can optionally cherry-pick unto the main branch. Almost there! |
6b37a01
to
43e67eb
Compare
4fbfaec
to
e481dec
Compare
839bcd3
to
a5c57ce
Compare
a5c57ce
to
b75d941
Compare
@@ -11,7 +11,7 @@ | |||
|
|||
require "functions" | |||
|
|||
MIN_BUNDLER_VERSION = "2.0.0" | |||
MIN_BUNDLER_VERSION = "2.1.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We actually rely on the Bundler v2.1.0 API for Bundler::URI, which was private in 2.0.x
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
6a4290b
to
332c3e0
Compare
b75d941
to
df1080b
Compare
🎉 Looks like all bundler tests are ✅ I would probably just merge and start the release over waiting for python to complete 👌 |
Depends on #3406
This modifies the
bundler_2_available
option so it no longer forces use of the bundler 2 native helpers but rather decides based on the lockfile if present.The primary change is to expand and test the
Dependabot::Bundler::Helper
to clearly specify scenarios such as defaulting to the newest when no Lockfile is present and failing over to v1 if a file is present but appears to pre-date theBUNDLED_WITH
notation.This also clarifies how Dependabot will behave if a new/malformed Gemfile.lock version were to appear.