Skip to content

Migrate from ESLint to Biome for code formatting and linting#12

Merged
mrousavy merged 16 commits into
mrousavy:mainfrom
bglgwyng:configure-biome
Jul 16, 2025
Merged

Migrate from ESLint to Biome for code formatting and linting#12
mrousavy merged 16 commits into
mrousavy:mainfrom
bglgwyng:configure-biome

Conversation

@bglgwyng

@bglgwyng bglgwyng commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@bglgwyng bglgwyng changed the title refactor: migrate from ESLint to Biome for code formatting and linting Migrate from ESLint to Biome for code formatting and linting Jul 9, 2025
@bglgwyng bglgwyng marked this pull request as draft July 11, 2025 16:36
@bglgwyng bglgwyng marked this pull request as ready for review July 11, 2025 16:36
@bglgwyng

bglgwyng commented Jul 12, 2025

Copy link
Copy Markdown
Contributor Author

@mrousavy Could you review this PR? This is blocking #11

@mrousavy

Copy link
Copy Markdown
Owner

did you add a github workflow/CI to make sure everything is linted?

@bglgwyng

bglgwyng commented Jul 13, 2025

Copy link
Copy Markdown
Contributor Author

I just added 'Run Nitrogen' workflow.

@mrousavy

Copy link
Copy Markdown
Owner

Cool, but please configure Biome to use 4 spaces. Then we can merge

@mrousavy

Copy link
Copy Markdown
Owner

So the Nitrogen CI fails because apparently Biome also formats files in nitrogen/generated- which it shouldn't do. Can you ignore that path? Then we're ready to merge

@bglgwyng

Copy link
Copy Markdown
Contributor Author

I think now it's finally ready.

@mrousavy

Copy link
Copy Markdown
Owner

It still formatted nitrogen/generated/shared/json/NitroImageViewConfig.json:
Screenshot 2025-07-15 at 22 23 48

@bglgwyng

Copy link
Copy Markdown
Contributor Author

Ah sorry. I just fixed it. I should have tested CI sooner.

bglgwyng#2

@mrousavy mrousavy merged commit e5942ab into mrousavy:main Jul 16, 2025
4 checks passed
@mrousavy

Copy link
Copy Markdown
Owner

awesome! thx

@bglgwyng bglgwyng deleted the configure-biome branch July 17, 2025 04:47
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