Skip to content

Commit

Permalink
ci: pack packages in PR tests
Browse files Browse the repository at this point in the history
  • Loading branch information
turadg committed May 15, 2024
1 parent d1a35d8 commit 01e4ac2
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions .github/workflows/test-all-packages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,23 @@ jobs:
- name: Install graphviz
run: sudo apt install -y graphviz

# This is now redundant with Lerna's --reject-cycles but keep it here so we have
# test coverage of this script we use for visualizing the dep graph.
- name: Check for cycles
run: scripts/check-dependency-cycles.sh

# install node_modules
- uses: ./.github/actions/restore-node

# Releasing SDK builds all packages in this order during `lerna publish`.
# Due to ambient types, certain build orders (from the dependency graph)
# may break the type resolution. Run this in the PR to find out before
# attempting to merge to master. This takes about 1min locally and since
# this job is about 30s in CI doing it here doesn't add to wall wait
# for CI resolution.
- name: Pack packages
run: yarn lerna exec --reject-cycles --concurrency 1 "npm pack"

##################
# Lint tests
# We run per package bc of https://github.com/typescript-eslint/typescript-eslint/issues/1192
Expand Down

0 comments on commit 01e4ac2

Please sign in to comment.