Force npm self-install in release workflow #258
Merged
Claude / Claude Code Review
completed
Apr 22, 2026 in 7m 49s
Code review found 1 potential issue
Found 4 candidates, confirmed 1. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 0 |
| 🟡 Nit | 0 |
| 🟣 Pre-existing | 1 |
| Severity | File:Line | Issue |
|---|---|---|
| 🟣 Pre-existing | .github/workflows/release.yml:345-351 |
Deprecated ::set-output syntax in charts-release job |
Annotations
Check notice on line 351 in .github/workflows/release.yml
claude / Claude Code Review
Deprecated ::set-output syntax in charts-release job
The charts-release job's 'Output new version' step uses the deprecated '::set-output' workflow command (disabled by GitHub in May 2023), so 'needs.charts-release.outputs.version' is always empty on modern runners. This is a pre-existing issue unrelated to this PR's change, but since this PR already touches release.yml it's a good time to fix line 176 to use 'echo "version=$(...)" >> $GITHUB_OUTPUT'.
Loading