Skip to content

Conversation

@bglgwyng
Copy link
Contributor

@bglgwyng bglgwyng commented Jul 9, 2025

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
Copy link
Contributor Author

bglgwyng commented Jul 12, 2025

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

@mrousavy
Copy link
Owner

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

@bglgwyng
Copy link
Contributor Author

bglgwyng commented Jul 13, 2025

I just added 'Run Nitrogen' workflow.

@mrousavy
Copy link
Owner

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

@mrousavy
Copy link
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
Contributor Author

I think now it's finally ready.

@mrousavy
Copy link
Owner

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

@bglgwyng
Copy link
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
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