Skip to content
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

Enforce use of Bundler >= 2.4 #186

Closed
wants to merge 1 commit into from
Closed

Enforce use of Bundler >= 2.4 #186

wants to merge 1 commit into from

Conversation

mcmire
Copy link
Collaborator

@mcmire mcmire commented Mar 25, 2023

We make use of install_if in our appraisals. However, if you aren't using Bundler 2.4 then you can't generate any appraisal gemfiles.

This commit enforces the correct version of Bundler by adding a check to the Gemfile itself. This isn't ideal, but we can't simply add gem "bundler", "~> 2.4" to the Gemfile because of the new-ish logic in Bundler which automatically switches to the version of Bundler specified by the BUNDLED WITH section in the lockfile. Unfortunately, we can't commit the lockfiles to the repo, so neither adding the gem line nor updating the BUNDLED WITH would work.

We make use of `install_if` in our appraisals. However, if you aren't
using Bundler 2.4 then you can't generate any appraisal gemfiles.

This commit enforces the correct version of Bundler by adding a check to
the Gemfile itself. This isn't ideal, but we can't simply add `gem
"bundler", "~> 2.4"` to the Gemfile because of the new-ish logic in
Bundler which [automatically switches to the version of Bundler
specified by the `BUNDLED WITH` section in the lockfile][1].
Unfortunately, we can't commit the lockfiles to the repo, so neither
adding the `gem` line nor updating the `BUNDLED WITH` would work.

[1]: https://bundler.io/blog/2022/01/23/bundler-v2-3.html
@mcmire
Copy link
Collaborator Author

mcmire commented Mar 26, 2023

Hmm. This might not be necessary after we remove old Rubies and Rails. Stand by...

@mcmire
Copy link
Collaborator Author

mcmire commented Mar 26, 2023

Yup, we don't need this right now. Closing due to #187.

@mcmire mcmire closed this Mar 26, 2023
@mcmire mcmire deleted the use-bundler-2-4 branch March 26, 2023 01:57
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.

1 participant