Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,11 @@ jobs:
with:
node-version: 20

- name: Install root dependencies
run: npm ci
# The root package.json declares zero runtime dependencies —
# the verify scripts only use built-in `node:*` modules — so
# there is nothing to install. Skip `npm ci` (which would
# otherwise fail hard because there is no root
# package-lock.json to consume).

- name: Run verify script
run: node scripts/verify.mjs
12 changes: 11 additions & 1 deletion .github/workflows/wasm-size.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,14 +67,24 @@ jobs:

- name: Comment on PR
uses: actions/github-script@v7
env:
# Pass the multi-line report through an env var rather than
# via `${{ steps... }}` substitution. Substitution inlines the
# report text directly into the JavaScript source before it
# runs, which breaks single-quoted string literals as soon as
# the report spans more than one line (or contains quotes,
# backticks, or backslashes). Reading from the env var keeps
# the report an opaque string at runtime.
REPORT: ${{ steps.sizes.outputs.report }}
with:
script: |
const marker = '<!-- wasm-size-report -->';
const report = process.env.REPORT || '(no report)';
const body = [
marker,
'## WASM Binary Sizes',
'',
'${{ steps.sizes.outputs.report }}',
report,
'',
`_Built from \`${context.sha.slice(0, 8)}\` on \`${context.ref.replace('refs/heads/', '')}\`_`,
].join('\n');
Expand Down
Loading