Skip to content

feat: migrate from StandardJS to ESLint with neostandard#98

Merged
mcollina merged 4 commits into
masterfrom
migrate-to-eslint-neostandard
Sep 21, 2025
Merged

feat: migrate from StandardJS to ESLint with neostandard#98
mcollina merged 4 commits into
masterfrom
migrate-to-eslint-neostandard

Conversation

@mcollina

Copy link
Copy Markdown
Owner

Summary

  • Replace StandardJS with ESLint and neostandard configuration
  • Update CI to run linting only on Node.js 20+ to ensure modern ESLint compatibility
  • Apply auto-fixes for modern JavaScript syntax (object shorthand)
  • Add development documentation in CLAUDE.md

Changes Made

  • Dependencies: Removed standard and snazzy, added eslint and neostandard
  • Configuration: Created eslint.config.js with neostandard preset
  • Scripts: Updated npm run lint to use ESLint
  • CI: Modified GitHub Actions to conditionally run linting on Node 20+
  • Code style: Applied ESLint auto-fixes (object method shorthand)
  • Documentation: Added CLAUDE.md for development guidance

Test Plan

  • All existing tests pass (226 tests, 100% coverage)
  • Linting runs successfully with no errors
  • CI configuration validates on multiple Node versions
  • Pre-commit hooks still work correctly

🤖 Generated with Claude Code

mcollina and others added 4 commits September 20, 2025 22:52
- Replace standard and snazzy with eslint and neostandard
- Update package.json scripts to use eslint
- Add eslint.config.js with neostandard configuration
- Update CI to run linting only on Node.js 20+
- Apply ESLint auto-fixes (object shorthand)
- Remove unnecessary eslint-disable in example.mjs
- Add CLAUDE.md for development guidance

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

Co-Authored-By: Claude <noreply@anthropic.com>
- Configure ESLint to disable object-shorthand rule which is not supported in Node.js 0.10
- Revert object property shorthand syntax to longhand format
- Maintain all other neostandard rules while ensuring legacy Node.js compatibility

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

Co-Authored-By: Claude <noreply@anthropic.com>
Add .npmrc with ignore-scripts=true to prevent install script failures
on legacy Node.js versions that don't support modern dependencies.

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

Co-Authored-By: Claude <noreply@anthropic.com>
Remove outdated StandardJS badge since we've migrated to ESLint with neostandard

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

Co-Authored-By: Claude <noreply@anthropic.com>
@mcollina mcollina merged commit c01e649 into master Sep 21, 2025
31 checks passed
@mcollina mcollina deleted the migrate-to-eslint-neostandard branch September 21, 2025 07:46
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.

1 participant