Skip to content

Commit c7dd4c5

Browse files
justin808claude
andcommitted
Fix hardcoded Node version and restore yalc command consistency
Critical fixes: 1. Use translated node-version from matrix instead of hardcoded '20' - Latest config will now correctly use Node 22 instead of Node 20 2. Restore yalc command to match other workflows: 'cd packages/react-on-rails && yalc publish' 3. Re-enable conditional yarn cache (disabled for Node 22 due to V8 bug) This ensures the matrix actually controls which Node version is used. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 7ecb26e commit c7dd4c5

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/examples.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -99,11 +99,11 @@ jobs:
9999
- name: Setup Node
100100
uses: actions/setup-node@v4
101101
with:
102-
node-version: 20
103-
# TODO: Re-enable yarn caching once Node.js V8 cache crash is fixed
104-
# Tracking: https://github.com/actions/setup-node/issues/1028
105-
# cache: yarn
106-
# cache-dependency-path: '**/yarn.lock'
102+
node-version: ${{ steps.translate-matrix.outputs.node-version }}
103+
# Disable cache for Node 22 due to V8 bug in 22.21.0
104+
# https://github.com/nodejs/node/issues/56010
105+
cache: ${{ steps.translate-matrix.outputs.node-version != '22' && 'yarn' || '' }}
106+
cache-dependency-path: '**/yarn.lock'
107107
- name: Print system information
108108
run: |
109109
echo "Linux release: "; cat /etc/issue
@@ -128,7 +128,7 @@ jobs:
128128
yarn install --no-progress --no-emoji ${{ matrix.dependency-level == 'latest' && '--frozen-lockfile' || '' }}
129129
sudo yarn global add yalc
130130
- name: yalc publish for react-on-rails
131-
run: yarn yalc publish
131+
run: cd packages/react-on-rails && yalc publish
132132
- name: Install Ruby Gems for package
133133
run: |
134134
bundle lock --add-platform 'x86_64-linux'

0 commit comments

Comments
 (0)