Skip to content

Commit dea2ba1

Browse files
committed
chore(ci): remove per-commit build artifact caching to reduce cache pressure
1 parent 2dc30ed commit dea2ba1

3 files changed

Lines changed: 2 additions & 40 deletions

File tree

.github/workflows/reusable-e2e.yml

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -142,30 +142,13 @@ jobs:
142142
env:
143143
SEGMENT_DOWNLOAD_TIMEOUT_MINS: 3
144144

145-
- name: Restore ui/dist cache
146-
uses: actions/cache@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0 https://github.com/actions/cache/commit/1bd1e32a3bdc45362d1e726936510720a7c30a57
147-
id: restore-ui-cache
148-
with:
149-
path: ./packages/ui/dist
150-
key: ${{ runner.os }}-ui-${{ inputs.commit }}
151-
152-
- name: Restore ${{ matrix.package }}/dist cache
153-
id: restore-package-cache
154-
uses: actions/cache@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0 https://github.com/actions/cache/commit/1bd1e32a3bdc45362d1e726936510720a7c30a57
155-
with:
156-
path: ./packages/${{ matrix.package }}/dist
157-
key: ${{ runner.os }}-${{ matrix.package }}-${{ inputs.commit }}
158-
env:
159-
SEGMENT_DOWNLOAD_TIMEOUT_MINS: 1
160-
161145
- name: Install packages
162146
if: steps.restore-cache.outputs.cache-hit != 'true' || steps.restore-cypress-cache.outputs.cache-hit != 'true'
163147
uses: ./.github/actions/install-with-retries
164148
with:
165149
skip-cypress-binary: false
166150

167151
- name: Build @aws-amplify/ui package
168-
if: steps.restore-ui-cache.outputs.cache-hit != 'true'
169152
run: yarn ui build
170153

171154
- name: Build react-core package
@@ -177,7 +160,6 @@ jobs:
177160
run: yarn react-core-notifications build
178161

179162
- name: Build ${{ matrix.package }} package
180-
if: steps.restore-package-cache.outputs.cache-hit != 'true'
181163
run: yarn ${{ matrix.package }} build
182164

183165
- name: Build react-notifications package

.github/workflows/reusable-setup-cache.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -61,12 +61,5 @@ jobs:
6161
with:
6262
# only install binary if cypress cache missed
6363
skip-cypress-binary: ${{ steps.restore-cypress-cache.outputs.cache-hit }}
64-
- name: Restore packages/ui/dist from Cache
65-
uses: actions/cache@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0 https://github.com/actions/cache/commit/1bd1e32a3bdc45362d1e726936510720a7c30a57
66-
id: restore-ui-cache
67-
with:
68-
path: ./packages/ui/dist
69-
key: ${{ runner.os }}-ui-${{ inputs.commit }}
7064
- name: Build ui package
71-
if: steps.restore-ui-cache.outputs.cache-hit != 'true'
7265
run: yarn ui build

.github/workflows/reusable-unit.yml

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -62,23 +62,14 @@ jobs:
6262
env:
6363
SEGMENT_DOWNLOAD_TIMEOUT_MINS: 3
6464

65-
- name: Restore ui/dist cache
66-
uses: actions/cache@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0 https://github.com/actions/cache/commit/1bd1e32a3bdc45362d1e726936510720a7c30a57
67-
id: restore-ui-cache
68-
with:
69-
path: ./packages/ui/dist
70-
key: ${{ runner.os }}-ui-${{ inputs.commit }}
71-
env:
72-
SEGMENT_DOWNLOAD_TIMEOUT_MINS: 1
73-
7465
- name: Install packages
7566
if: steps.restore-cache.outputs.cache-hit != 'true'
7667
uses: ./.github/actions/install-with-retries
7768
with:
7869
skip-cypress-binary: true # unit test doesn't need cypress
7970

8071
- name: Build ui package for downstream dependencies
81-
if: ${{ steps.restore-ui-cache.outputs.cache-hit != 'true' && matrix.package != 'ui' }}
72+
if: ${{ matrix.package != 'ui' }}
8273
run: yarn ui build
8374

8475
- name: Build react-core package
@@ -135,8 +126,4 @@ jobs:
135126
matrix.package == 'react-storage'
136127
run: yarn ${{ matrix.package }} size
137128

138-
- name: Cache ${{ matrix.package }}/dist
139-
uses: actions/cache@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0 https://github.com/actions/cache/commit/1bd1e32a3bdc45362d1e726936510720a7c30a57
140-
with:
141-
path: ./packages/${{ matrix.package }}/dist
142-
key: ${{ runner.os }}-${{ matrix.package }}-${{ inputs.commit }}
129+

0 commit comments

Comments
 (0)