Skip to content

Commit e443bf2

Browse files
facundofariasclaude
andcommitted
ci: Skip linting on Node 16.x due to ESLint 9 requirements
ESLint 9.x requires Node.js 18.18+ (uses structuredClone internally). Since we support Node 16+ for runtime but ESLint is a dev tool, skip linting on Node 16.x in CI while keeping it for 18.x, 20.x, and 22.x. Changes: - Skip linting step for Node 16.x in CI workflow - Document development requirements (Node 18+) in README - Update CLAUDE.md to clarify runtime vs development requirements The MCP server still runs perfectly on Node 16+, but development tools (ESLint, etc.) require Node 18+. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent f7acae8 commit e443bf2

File tree

3 files changed

+8
-1
lines changed

3 files changed

+8
-1
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ jobs:
2929
run: npm ci
3030

3131
- name: Run linter
32+
if: matrix.node-version != '16.x'
3233
run: npm run lint
3334

3435
- name: Run type check

CLAUDE.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,4 +159,6 @@ For stdio testing, use absolute path to `dist/stdio.js` in client configs to avo
159159

160160
## Important: Node Version
161161

162-
Requires Node.js >=16.0.0 (specified in package.json engines). The server uses node-fetch for HTTP requests to ensure compatibility with Node 16+, which is commonly used by Claude Desktop/Code installations.
162+
**Runtime**: Requires Node.js >=16.0.0 (specified in package.json engines). The server uses node-fetch for HTTP requests to ensure compatibility with Node 16+, which is commonly used by Claude Desktop/Code installations.
163+
164+
**Development**: Requires Node.js >=18.0.0 for development tools (ESLint 9.x requires Node 18.18+). If you're on Node 16, you can still build and test, but skip linting with `npm run build && npm run type-check`.

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,10 @@ Once configured, you can ask Claude to interact with DeployHQ:
139139

140140
## 🔧 Local Development
141141

142+
**Development Requirements:**
143+
- Node.js 18+ (for ESLint 9.x and other dev tools)
144+
- Runtime supports Node.js 16+
145+
142146
### 1. Clone the repository
143147

144148
```bash

0 commit comments

Comments
 (0)