Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace eslint and prettier with Biome #4307

Merged
merged 3 commits into from
Dec 31, 2024
Merged

Conversation

frjo
Copy link
Member

@frjo frjo commented Dec 31, 2024

Fixes #4295

Keeping stylelint for sass files since biome doesn't support them, yet.

The biome configuration is set up to induce minal changes to current codebase.

With time we should move towards using biome "recommended" configuration as is.

@frjo frjo changed the title Maintenance/implement biome Replace eslint and prettier with Biome Dec 31, 2024
@frjo frjo added Type: Maintenance Type: Patch Mini change, used in release drafter labels Dec 31, 2024
@frjo frjo force-pushed the maintenance/implement-biome branch from 1e0e92b to b9c7b4e Compare December 31, 2024 11:52
@frjo frjo requested a review from theskumar December 31, 2024 12:00
Copy link
Member

@theskumar theskumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, literally you have the config that doesn't update any files. 💯

@frjo frjo merged commit 0d72e1c into main Dec 31, 2024
7 checks passed
@frjo frjo deleted the maintenance/implement-biome branch March 11, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace styelint, eslint and prettier with biomejs
2 participants