-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Support repositories with default branch not named master #4224
Conversation
52b4678
to
cf8753d
Compare
If I understand this well, this also fixes "bug" when |
Now that you mention it, yeah, it sounds like that's what the old code was doing! |
cf8753d
to
acb6f91
Compare
It should be ready now unless CI proves me wrong! |
acb6f91
to
de963bc
Compare
CI got a flaky issue on jruby which I'm trying to address at #4235. I'll rebasing to see how this plays with the latest CI additions, and then merge. |
Bundler 3 no longer prints these messages, but we can force them with `--verbose`.
… master Without having to pass an explicit `:branch` argument. Now git source string representation no longer shows branch information unless the repository has already been cloned and we know what the default branch is.
de963bc
to
c869343
Compare
Support repositories with default branch not named master (cherry picked from commit 6e032a9)
What was the end-user or developer problem that led to this PR?
Bundler doesn't support non-master default branches, so you can't currently point to, for example, simplecov's repo, unless you add an explicit
branch: => "main"
to your git sources configuration.Bundler should just work in this case without having to make the default branch explicit.
What is your fix for the problem, implemented in this PR?
My fix is to make sure the code doesn't assume a specific branch until we have actually cloned the repository. This means that when a git sources is printed, it no longer includes "master" if the source is only present in the
Gemfile
and has not yet been installed.Fixes #4009.
Make sure the following tasks are checked