Skip to content

Commit 97f5d73

Browse files
authored
fix(ci): correct cache key path for west manifest (#832)
hashFiles('manifest-dir/west.yml') never matched any file, so the cache key was always identical regardless of ZMK version changes. This meant upgrading the fork revision in config/west.yml would silently use stale cached modules. Fix both Legacy and Clique job cache keys to point at the actual manifest path. Made-with: Cursor
1 parent 962cb1d commit 97f5d73

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/build.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
zephyr/
3232
bootloader/
3333
zmk/
34-
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('manifest-dir/west.yml') }}
34+
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('config/west.yml') }}
3535
restore-keys: |
3636
${{ runner.os }}-build-${{ env.cache-name }}-
3737
${{ runner.os }}-build-
@@ -87,7 +87,7 @@ jobs:
8787
zephyr/
8888
bootloader/
8989
zmk/
90-
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('manifest-dir/west.yml') }}
90+
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('config/west.yml') }}
9191
restore-keys: |
9292
${{ runner.os }}-build-${{ env.cache-name }}-
9393
${{ runner.os }}-build-

0 commit comments

Comments
 (0)