Skip to content

Commit b9618e8

Browse files
authored
ci: remove redundant Pack step (#1115)
## 📜 Description Remove redundant `Pack` step. ## 💡 Motivation and Context It looks like I added this step for debugging purposes and forgot to remove it. There is no sense to keep it, because `scripts/size.js` already uses `npm pack --json` command so there is no reason to run the command twice. ## 📢 Changelog <!-- High level overview of important changes --> <!-- For example: fixed status bar manipulation; added new types declarations; --> <!-- If your changes don't affect one of platform/language below - then remove this platform/language --> ### CI - remove `Pack` command from CI size-diff job. ## 🤔 How Has This Been Tested? Tested manually via this PR. ## 📸 Screenshots (if appropriate): <img width="965" height="622" alt="image" src="https://github.com/user-attachments/assets/0fad211c-b58b-458d-8f2b-6b39ef2652fb" /> ## 📝 Checklist - [x] CI successfully passed - [x] I added new mocks and corresponding unit-tests if library API was changed
1 parent 2a3bcee commit b9618e8

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

.github/workflows/size-diff-collect.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,6 @@ jobs:
3737
- name: Install dependencies
3838
run: yarn install --frozen-lockfile
3939

40-
- name: Pack
41-
run: npm pack --json
42-
4340
- name: Calculate size of package (current)
4441
id: new-size
4542
run: echo "NEW_SIZE=$(node scripts/size.js)" >> "$GITHUB_OUTPUT"

0 commit comments

Comments
 (0)