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

Force bundler to the latest version on Ruby 3.1 #510

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Fryguy
Copy link
Member

@Fryguy Fryguy commented Nov 21, 2024

We noticed that our cache sizes were growing enormously, namely in the git repositories, where the git directory is not git gc'd. The bundler shipped with Ruby 3.1, bundler 2.3.27, exhibits the problem, whereas newer bundlers > 2.4.0 seem to have mitigated the problem because of the underlying change to how git respositories are shallow clones.

@agrare Please review.

We noticed that our cache sizes were growing enormously, namely in the
git repositories, where the git directory is not git gc'd. The bundler
shipped with Ruby 3.1, bundler 2.3.27, exhibits the problem, whereas
newer bundlers > 2.4.0 seem to have mitigated the problem because of
the underlying change to how git respositories are shallow clones.
@@ -37,6 +37,7 @@ jobs:
uses: ruby/setup-ruby@v1
with:
ruby-version: "${{ matrix.ruby-version }}"
bundler: "${{ matrix.ruby-version == '3.1' && 'latest' || 'default' }}"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alternatively, I could just set it to latest for everything, but I worry that we won't follow up with a removal when we drop 3.1. This way we can't miss it. Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants