Skip to content

Conversation

@nurazon59
Copy link
Contributor

I started working on #295 and noticed there's no formatterconfigured.
This made it difficult to ensure consistent code style.

This PR adds Prettier for formatting

Let me know if you have any preferences on the configuration - happy to adjust.

Copy link
Collaborator

@Almenon Almenon 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, thanks. The only request I have is to ignore the package-lock.json file, that's autogenerated so no point in prettier trying to format it.

https://prettier.io/docs/ignore

Edit: Actually, I can add this on my own. Approving!

@Almenon Almenon merged commit ea826d0 into extrabacon:master Dec 6, 2025
@Almenon
Copy link
Collaborator

Almenon commented Dec 6, 2025

also note there's more work if you want to take it on: #317

@nurazon59
Copy link
Contributor Author

Thank you for the review and for merging this.
I will take a look at #317 next.

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