Skip to content

Commit bcce9a1

Browse files
justin808claude
andcommitted
Remove unnecessary bundle lock --add-platform from CI workflow
The x86_64-linux platform is already present in all Gemfile.lock files, so running 'bundle lock --add-platform' is unnecessary and was causing the lockfile to be modified during CI runs, particularly on Ruby 2.7. This fixes the check_react_and_ujs (2.7) test failure. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent bd0aac3 commit bcce9a1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/ruby.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ jobs:
4747
path: vendor/bundle
4848
key: root-gem-cache-${{ hashFiles('Gemfile.lock') }}
4949
- name: Install Ruby Gems
50-
run: bundle lock --add-platform 'x86_64-linux' && bundle check --path=vendor/bundle || bundle _2.4.9_ install --path=vendor/bundle --jobs=4 --retry=3
50+
run: bundle check --path=vendor/bundle || bundle _2.4.9_ install --path=vendor/bundle --jobs=4 --retry=3
5151
- run: yarn
5252
- run: bundle exec rake react:update
5353
- run: bundle exec rake ujs:update
@@ -114,7 +114,7 @@ jobs:
114114
path: test/dummy/vendor/bundle
115115
key: dummy-app-gem-cache-${{ hashFiles(format('{0}/gemfiles/{1}.gemfile.lock', github.workspace, matrix.gemfile)) }}
116116
- name: Install Ruby Gems for dummy app
117-
run: bundle lock --add-platform 'x86_64-linux' && bundle check --path=test/dummy/vendor/bundle || bundle _2.4.9_ install --frozen --path=test/dummy/vendor/bundle --jobs=4 --retry=3
117+
run: bundle check --path=test/dummy/vendor/bundle || bundle _2.4.9_ install --frozen --path=test/dummy/vendor/bundle --jobs=4 --retry=3
118118
- run: cd test/dummy && yalc add react_ujs && ${{ matrix.js_package_manager.installer }} install
119119
- run: bundle exec rake test
120120
env:

0 commit comments

Comments
 (0)