Skip to content

Commit 7b84a75

Browse files
committed
wip
1 parent a7a8e1b commit 7b84a75

File tree

1 file changed

+1
-11
lines changed

1 file changed

+1
-11
lines changed

.github/workflows/ruby.yml

+1-11
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,6 @@ jobs:
7979
env:
8080
# $BUNDLE_GEMFILE must be set at the job level, so it is set for all steps
8181
BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
82-
LOCKFILE: '${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile.lock'
83-
LOCKFILE_HASH: hashFiles('${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile.lock')
8482
# Workaround b/c upgrading Minitest broke some mocking expectations
8583
# having to do with automatic kwarg splatting
8684
MT_KWARGS_HACK: 1
@@ -107,20 +105,12 @@ jobs:
107105
with:
108106
bundler: 2.4.9
109107
ruby-version: ${{ matrix.ruby }}
110-
- run: echo hash=${LOCKFILE_HASH} >> $GITHUB_ENV
111-
- run: echo lockfilepath=${LOCKFILE} >> $GITHUB_ENV
112108
- run: ./test/bin/create-fake-js-package-managers ${{ matrix.js_package_manager.installer }}
113-
- run: echo ${{ hashFiles('${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile.lock') }}
114-
- run: echo ${{ hashFiles('/home/runner/work/react-rails/react-rails/gemfiles/base.gemfile.lock') }}
115-
- run: echo ${{ hashFiles(${{ LOCKFILE }}) }}
116-
- run: echo ${{ hashFiles(${{ env.lockfilepath }}) }}
117-
- run: echo '${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile.lock'
118-
- run: ls -l '${{ github.workspace }}/gemfiles/'
119109
- name: Save dummy app ruby gems to cache
120110
uses: actions/cache@v3
121111
with:
122112
path: test/dummy/vendor/bundle
123-
key: dummy-app-gem-cache-${{ env.hash }}
113+
key: dummy-app-gem-cache-${{ hashFiles(format('{0}/gemfiles/{1}.gemfile.lock', github.workspace, matrix.gemfile)) }}
124114
- name: Install Ruby Gems for dummy app
125115
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
126116
- run: cd test/dummy && yalc add react_ujs && ${{ matrix.js_package_manager.installer }} install

0 commit comments

Comments
 (0)