Skip to content

Commit d5ec09a

Browse files
committed
fix(ci): use npm for markdownlint-cli installation
markdownlint-cli is a Node.js package, not Python. Also removed || true to ensure CI fails on actual lint errors.
1 parent b4fd6ea commit d5ec09a

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

.github/workflows/ci.yml

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,23 +13,22 @@ jobs:
1313
steps:
1414
- uses: actions/checkout@v4
1515

16-
- name: Set up Python 3.11
17-
uses: actions/setup-python@v5
16+
- name: Set up Node.js
17+
uses: actions/setup-node@v4
1818
with:
19-
python-version: "3.11"
19+
node-version: "20"
2020

2121
- name: Install dependencies
2222
run: |
23-
pip install markdownlint-cli
23+
npm install -g markdownlint-cli @mermaid-js/mermaid-cli
2424
2525
- name: Lint markdown files
2626
run: |
27-
markdownlint docs/**/*.md README.md CLAUDE.md || true
27+
markdownlint docs/**/*.md README.md CLAUDE.md
2828
2929
- name: Validate Mermaid diagrams
3030
run: |
31-
npm install -g @mermaid-js/mermaid-cli
32-
./scripts/validate-mermaid.sh || true
31+
./scripts/validate-mermaid.sh
3332
3433
- name: Check documentation structure
3534
run: |

0 commit comments

Comments
 (0)