Skip to content

Conversation

cressie176
Copy link
Collaborator

Starting point for applying lint rules

cressie176 and others added 3 commits September 3, 2025 22:56
- Enable Biome linting with all failing rules explicitly disabled
- Configure 18 different lint rules across 4 categories:
  * complexity: 6 rules (noCommaOperator, noUselessCatch, useArrowFunction, useOptionalChain, noArguments, useLiteralKeys)
  * correctness: 7 rules (noUnusedFunctionParameters, noUnusedVariables, noInnerDeclarations, useParseIntRadix, noSwitchDeclarations, noInvalidUseBeforeDeclaration, noPrecisionLoss)
  * style: 4 rules (useConst, useNodejsImportProtocol, useTemplate, useExponentiationOperator)
  * suspicious: 8 rules (noRedundantUseStrict, noAssignInExpressions, noAsyncPromiseExecutor, noDoubleEquals, noGlobalIsNan, noRedeclare, noGlobalIsFinite, noPrototypeBuiltins)
- Add npm run lint script for easy linting
- Lint now passes cleanly with no errors or warnings
- Individual rules can be re-enabled incrementally in future PRs

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- **BREAKING**: Update minimum Node.js requirement from v16 to v18
- Update Makefile test versions to current LTS and stable releases:
  * Node.js 18.1.0 → 18.20.5 (LTS maintenance)
  * Node.js 20.10.0 → 20.18.0 (LTS active)
  * Node.js 22.14.0 → 22.11.0 (current)
  * Add Node.js 24.7.0 (preview of next LTS)
- All 271 tests continue to pass with Node.js v18

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@cressie176 cressie176 changed the title Add comprehensive Biome linting baseline configuration Modernisation 2 - Add comprehensive Biome linting baseline configuration Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants