Skip to content

Commit

Permalink
Remove bundlesize from list of devDependencies
Browse files Browse the repository at this point in the history
bundlesize has some indirect dependencies which require extra build
time. Currently, it is only used for CI, so the extra time required to
build/install is unnecessary for most use cases.

Instead of installing bundlesize alongside Auspice, install it through
npx where it is used. Installing through npx installs to a folder in the
npm cache.
  • Loading branch information
victorlin committed Mar 14, 2023
1 parent 57c27b3 commit 7ae198c
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 1,198 deletions.
6 changes: 5 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,11 @@ jobs:
with:
node-version: ${{ matrix.node }}
- run: npm ci
- run: npx bundlesize
# Node.js 14 comes with NPM 6 which auto-installs packages by default.
- if: ${{ matrix.node == 14 }}
run: npx bundlesize
- if: ${{ matrix.node != 14 }}
run: npx --yes bundlesize
publish:
if: ${{ github.ref == 'refs/heads/release' }}
needs: [build, unit-test, smoke-test, lint]
Expand Down
Loading

0 comments on commit 7ae198c

Please sign in to comment.