Bump pipeline action versions - #1770
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates CI/workflow dependencies by upgrading the Node.js runtime used in GitHub Actions and pinning key workflow actions to specific commit SHAs for supply-chain hardening.
Changes:
- Updated workflows to use Node.js
24.15.0(from 22). - Pinned
actions/checkout,actions/setup-node, andpnpm/action-setupto specific commit SHAs instead of version tags. - Updated workflows to use the new pinned action references consistently across CI/E2E/benchmark jobs.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/sanity.yml | Pins actions/checkout to a commit SHA in the sanity workflow. |
| .github/workflows/extract-api.yml | Pins checkout/pnpm/setup-node and bumps Node to 24.15.0 for API extraction. |
| .github/workflows/e2e.yml | Pins checkout/pnpm/setup-node and bumps Node to 24.15.0 across E2E jobs. |
| .github/workflows/codeql.yml | Pins actions/checkout to a commit SHA in the CodeQL workflow. |
| .github/workflows/ci.yml | Pins checkout/pnpm/setup-node and bumps Node to 24.15.0 across CI jobs. |
| .github/workflows/changeset-release.yml | Bumps release workflow Node version to 24.15.0. |
| .github/workflows/benchmark-tree-widget.yml | Pins checkout/pnpm/setup-node and bumps Node to 24.15.0 for benchmarks. |
| .github/workflows/benchmark-pr-tree-widget.yml | Pins checkout/pnpm/setup-node and bumps Node to 24.15.0 for PR benchmarks. |
Suppressed comments (6)
.github/workflows/e2e.yml:79
- The pnpm setup step no longer pins a pnpm version. That makes CI runs dependent on whatever pnpm version the action installs by default, which can drift over time. Since the repo already declares
packageManager: pnpm@11.20.0inpackage.json, consider explicitly pinning the action'sversioninput to match for reproducible installs.
- name: Install pnpm
uses: pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093
.github/workflows/e2e.yml:123
- The pnpm setup step no longer pins a pnpm version. That makes CI runs dependent on whatever pnpm version the action installs by default, which can drift over time. Since the repo already declares
packageManager: pnpm@11.20.0inpackage.json, consider explicitly pinning the action'sversioninput to match for reproducible installs.
- name: Install pnpm
uses: pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093
.github/workflows/ci.yml:37
pnpm/action-setupis pinned to a commit but the pnpm version isn't pinned. If the action default changes, CI may start using a different pnpm version unexpectedly. Since the repo declarespackageManager: pnpm@11.20.0inpackage.json, consider settingwith: version: 11.20.0here for reproducibility.
- name: Setup pnpm
uses: pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093
.github/workflows/ci.yml:59
pnpm/action-setupis pinned to a commit but the pnpm version isn't pinned. If the action default changes, CI may start using a different pnpm version unexpectedly. Since the repo declarespackageManager: pnpm@11.20.0inpackage.json, consider settingwith: version: 11.20.0here for reproducibility.
- name: Setup pnpm
uses: pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093
.github/workflows/ci.yml:80
pnpm/action-setupis pinned to a commit but the pnpm version isn't pinned. If the action default changes, CI may start using a different pnpm version unexpectedly. Since the repo declarespackageManager: pnpm@11.20.0inpackage.json, consider settingwith: version: 11.20.0here for reproducibility.
- name: Setup pnpm
uses: pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093
.github/workflows/ci.yml:110
pnpm/action-setupis pinned to a commit but the pnpm version isn't pinned. If the action default changes, CI may start using a different pnpm version unexpectedly. Since the repo declarespackageManager: pnpm@11.20.0inpackage.json, consider settingwith: version: 11.20.0here for reproducibility.
- name: Setup pnpm
uses: pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
Tree-Widget benchmark
| Benchmark suite | Current: ec68700 | Previous: 8df07d4 | Deviation | Status |
|---|---|---|---|---|
models tree creates initial filtered view for 50k target items |
752 ms |
1071 ms |
-29.79% |
✅ |
models tree creates initial filtered view for 50k target items (P95 of main thread blocks) |
92 ms |
95 ms |
-3.16% |
〰️ |
models tree validates categories visibility for imodel with 50k categories |
6107 ms |
12371 ms |
-50.63% |
✅ |
models tree validates categories visibility for imodel with 50k categories (P95 of main thread blocks) |
72 ms |
85 ms |
-15.29% |
〰️ |
models tree changing model visibility changes visibility for 50k elements |
2313 ms |
3371 ms |
-31.39% |
✅ |
models tree changing model visibility changes visibility for 50k elements (P95 of main thread blocks) |
0 ms |
123 ms |
-100% |
✅ |
models tree changing category visibility changes visibility for 50k elements |
2343 ms |
3344 ms |
-29.93% |
✅ |
models tree changing category visibility changes visibility for 50k elements (P95 of main thread blocks) |
0 ms |
23 ms |
-100% |
〰️ |
models tree changing per-model-category override changes visibility for 50k elements |
2330 ms |
3301 ms |
-29.42% |
✅ |
models tree changing per-model-category override changes visibility for 50k elements (P95 of main thread blocks) |
0 ms |
0 ms |
0% |
🟰 |
models tree changing element visibility changes only parent nodes visibility with 50k elements |
3872 ms |
4669 ms |
-17.07% |
✅ |
models tree changing element visibility changes only parent nodes visibility with 50k elements (P95 of main thread blocks) |
90 ms |
89 ms |
1.12% |
〰️ |
categories tree creates initial filtered view for 50k items |
1606 ms |
1515 ms |
6.01% |
〰️ |
categories tree creates initial filtered view for 50k items (P95 of main thread blocks) |
605 ms |
303 ms |
99.67% |
🚨 |
categories tree changing definition container visibility changes visibility for 50k subCategories |
2401 ms |
2811 ms |
-14.59% |
✅ |
categories tree changing definition container visibility changes visibility for 50k subCategories (P95 of main thread blocks) |
38 ms |
57 ms |
-33.33% |
〰️ |
categories tree changing definition container visibility changes visibility for 50k categories |
4889 ms |
5886 ms |
-16.94% |
✅ |
categories tree changing definition container visibility changes visibility for 50k categories (P95 of main thread blocks) |
101 ms |
126 ms |
-19.84% |
〰️ |
This comment was automatically generated by workflow using github-action-benchmark.
aruniverse
approved these changes
Aug 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
24.15.0.6.0.26.4.06.0.8