feat: migrate from ESLint/Prettier to Biome #1701
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rmatting - Add Biome as a modern, fast alternative to ESLint + Prettier - Configure Biome to match existing code style (spaces, semicolons, single quotes) - Update npm scripts to use Biome commands (format, lint, lint:fix) - Remove ESLint and Prettier dependencies and config files - Configure Biome to ignore proto/, mocha/, and config files - Disable style rules that would require massive code changes - Keep only essential linting rules for code quality Performance: 10-100x faster (202 files in 33ms vs several seconds) BREAKING CHANGE: ESLint and Prettier are no longer used. Use 'npm run lint' and 'npm run format' with Biome instead.
- Apply Biome's opinionated formatting across the codebase - Updates include: - Closing parentheses moved to same line for arrow functions - Multi-parameter constructors formatted on separate lines - Unnecessary object key quotes removed - Better string concatenation indentation - These changes improve code readability and consistency
- Convert RegExp constructors to regex literals (better performance) - Disable noUselessTernary and noApproximativeNumericConstant rules - The ternary expressions are intentional for clarity - The PI test is checking a specific value, not Math.PI - All tests now pass with npm run lint
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.