Commit c66d59a
fix: configure setup-dependencies for SDK repo (#1503)
## Summary
- Removes the separate Node.js setup step (now handled by
shared-actions/setup-dependencies with `registry-url` for npm OIDC)
- Disables unused tools in setup-dependencies: foundry, helm, helm-docs,
python, chart-testing
This reduces CI time by not installing tools the SDK repo doesn't need.
## Test plan
- [ ] Verify CI passes
- [ ] Verify npm publish works with OIDC
🤖 Generated with [Claude Code](https://claude.com/claude-code)
<!-- This is an auto-generated description by cubic. -->
---
## Summary by cubic
Switch CI to shared-actions/setup-dependencies for Node and npm OIDC,
remove the standalone setup-node step, and align .node-version to 24.
Disable unused tools to cut CI time.
- **Refactors**
- Set runner: github for setup-dependencies; updated .node-version to
24.
- Disabled tools: foundry, helm, helm-docs, python, chart-testing.
<sup>Written for commit 3a06e87.
Summary will update on new commits.</sup>
<!-- End of auto-generated description by cubic. -->
---------
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>1 parent 5adc63e commit c66d59a
2 files changed
Lines changed: 7 additions & 9 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
121 | | - | |
122 | | - | |
123 | | - | |
124 | | - | |
125 | | - | |
126 | | - | |
127 | | - | |
128 | | - | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
129 | 127 | | |
130 | 128 | | |
131 | 129 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
0 commit comments