Skip to content

Commit c1e12dd

Browse files
sjnimsclaude
andcommitted
chore: eliminate workflow dependencies and optimize CI
- Remove setup-node from claude.yml (action bundles Node runtime) - Replace npm install with npx in claude-pr-review.yml Reduces setup overhead and eliminates dependency management. Co-Authored-By: Claude Sonnet 4.5 (1M context) <noreply@anthropic.com>
1 parent 367d8fb commit c1e12dd

2 files changed

Lines changed: 3 additions & 16 deletions

File tree

.github/workflows/claude-pr-review.yml

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,14 +33,6 @@ jobs:
3333
with:
3434
fetch-depth: 1
3535

36-
- name: Setup Node.js
37-
uses: actions/setup-node@395ad3262231945c25e8478fd5baf05154b1d79f # v6.1.0
38-
with:
39-
node-version: "20"
40-
41-
- name: Install markdownlint-cli
42-
run: npm install -g markdownlint-cli
43-
4436
- name: Review PR with Claude
4537
uses: anthropics/claude-code-action@1b8ee3b94104046d71fde52ec3557651ad8c0d71 # v1.0.29
4638
with:
@@ -56,7 +48,7 @@ jobs:
5648
5749
## Instructions
5850
1. Get the PR diff to understand what changed
59-
2. If any `.md` files were changed, run `markdownlint --config .markdownlint.json <files>` to check for style issues
51+
2. If any `.md` files were changed, run `npx markdownlint-cli2 <files>` to check for style issues
6052
3. Review the changes against the criteria below
6153
4. Post a summary comment with your findings
6254
@@ -69,7 +61,7 @@ jobs:
6961
- **Hooks** (`hooks/hooks.json`): Validate event types and matcher patterns.
7062
7163
### Markdown Quality (run markdownlint)
72-
Run `markdownlint --config .markdownlint.json` on changed `.md` files. Key rules enforced:
64+
Run `npx markdownlint-cli2` on changed `.md` files. Key rules enforced:
7365
- ATX-style headers (`#` not underlines)
7466
- Dash-style lists (`-` not `*` or `+`)
7567
- 2-space indentation for nested lists
@@ -90,4 +82,4 @@ jobs:
9082
Be constructive and helpful. Focus on significant issues, not nitpicks.
9183
If the PR looks good, say so briefly - don't invent problems.
9284
claude_args: |
93-
--allowedTools "Bash(gh pr:*),Bash(markdownlint:*),Read,Glob,Grep,mcp__github_inline_comment__create_inline_comment"
85+
--allowedTools "Bash(gh pr:*),Bash(npx markdownlint-cli2:*),Read,Glob,Grep,mcp__github_inline_comment__create_inline_comment"

.github/workflows/claude.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,6 @@ jobs:
5151
with:
5252
fetch-depth: 1
5353

54-
- name: Setup Node.js
55-
uses: actions/setup-node@395ad3262231945c25e8478fd5baf05154b1d79f # v6.1.0
56-
with:
57-
node-version: "20"
58-
5954
- name: Run Claude Code
6055
id: claude
6156
uses: anthropics/claude-code-action@1b8ee3b94104046d71fde52ec3557651ad8c0d71 # v1.0.29

0 commit comments

Comments
 (0)