Skip to content

Commit 9c67896

Browse files
committed
small cleanup and improvements
1 parent 4fa3cc4 commit 9c67896

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

.github/workflows/build.yml

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,6 @@ concurrency:
2828
env:
2929
HEAD_COMMIT: ${{ github.event.inputs.commit || github.sha }}
3030

31-
# WARNING: this disables cross os caching as ~ and
32-
# github.workspace evaluate to differents paths
3331
CACHED_DEPENDENCY_PATHS: |
3432
${{ github.workspace }}/node_modules
3533
${{ github.workspace }}/packages/*/node_modules
@@ -38,15 +36,10 @@ env:
3836
3937
# DEPENDENCY_CACHE_KEY: can't be set here because we don't have access to yarn.lock
4038

41-
# WARNING: this disables cross os caching as ~ and
42-
# github.workspace evaluate to differents paths
43-
# packages/utils/cjs and packages/utils/esm: Symlinks to the folders inside of `build`, needed for tests
44-
CACHED_BUILD_PATHS: |
39+
BUILD_PATHS: |
4540
${{ github.workspace }}/dev-packages/*/build
4641
${{ github.workspace }}/packages/*/build
47-
${{ github.workspace }}/packages/*/lib
48-
${{ github.workspace }}/packages/ember/*.d.ts
49-
${{ github.workspace }}/packages/gatsby/*.d.ts
42+
${{ github.workspace }}/packages/*.d.ts
5043
5144
# upload-artifact globs drop the path through the first `*` (see upload-artifact
5245
# README). Tarballs therefore land in the artifact as <pkg>/*.tgz; download
@@ -65,6 +58,7 @@ env:
6558
jobs:
6659
job_get_metadata:
6760
uses: ./.github/workflows/ci-metadata.yml
61+
name: Get CI Metadata
6862
with:
6963
head_commit: ${{ github.event.inputs.commit || github.sha }}
7064
permissions:
@@ -124,7 +118,7 @@ jobs:
124118
uses: actions/upload-artifact@v7
125119
with:
126120
name: build-output
127-
path: ${{ env.CACHED_BUILD_PATHS }}
121+
path: ${{ env.BUILD_PATHS }}
128122
retention-days: 4
129123
compression-level: 6
130124
overwrite: true

0 commit comments

Comments
 (0)