Skip to content

Commit f80974f

Browse files
kanwalpreetdclaude
andcommitted
socket-scan: address review comments
- Make the Node 24 env-var comment date-agnostic (2026-06-16 has passed) - Remove the unused Corepack step; this repo has no package.json, so it only ever shimmed yarn/pnpm that are never invoked - Replace dtolnay/rust-toolchain with `rustup update`, matching the convention in this repo's other Rust workflows and dropping a third-party action from a security workflow Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent d303fc6 commit f80974f

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

.github/workflows/socket-scan.yml

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -35,23 +35,18 @@ permissions:
3535

3636
env:
3737
# Force JS-based GitHub actions (actions/checkout, actions/setup-*, etc.) to
38-
# use Node 24 instead of the soon-to-be-deprecated Node 20. Safe to remove
39-
# after 2026-06-16 (when Node 24 becomes the default and this becomes a no-op).
38+
# use Node 24 instead of Node 20. Remove this once the runner default is Node 24.
4039
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true
4140

4241
jobs:
4342
socket-scan:
4443
runs-on: ubuntu-latest
4544
steps:
4645
- uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
47-
- uses: dtolnay/rust-toolchain@e97e2d8cc328f1b50210efc529dca0028893a2d9 # v1
48-
with:
49-
toolchain: "1.86.0"
46+
- run: rustup update
5047
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
5148
with:
5249
node-version: "24.18.0"
53-
- name: Enable Corepack (yarn/pnpm per repo packageManager)
54-
run: corepack enable
5550

5651
- name: Install Socket CLI
5752
run: npm install -g socket

0 commit comments

Comments
 (0)