-
Notifications
You must be signed in to change notification settings - Fork 206
ruby 3.2.6, bundler 2.6.2, add checksums to lockfile, ci simplification #3544
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
Conversation
df479cf
to
ef3fcce
Compare
In favor but circle seems unhappy? |
@katzj yeah, I was struggling with Circle on this 😭 Might pick it up again later today to retry |
f08aa6e
to
d1e4cd2
Compare
63aed2f
to
6664fa4
Compare
837bee3
to
530bd11
Compare
|
||
executors: | ||
# TODO: we don't need ES/PG/Redis for "lint" job, so we could create another | ||
# stripped-down executor if we want? | ||
libraries: | ||
working_directory: ~/libraries | ||
docker: | ||
- image: ruby:3.2.5-slim-bullseye | ||
- image: cimg/ruby:3.2-node |
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.
I think we just needed the custom debian packages below for node, so this switches to the node image
command: | | ||
gem -v | ||
bundle -v | ||
bundle config path vendor/bundle |
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.
this is already set by ruby/install-deps
checksums announcement: https://bundler.io/blog/2024/12/19/bundler-v2-6.html
the checksums were added manually to Gemfile.lock via
bundle lock --add-checksums
Update bundler to 2.6.1+ locally
bundle update --bundler