Skip to content

chore: simplify prettier config#528

Merged
aslilac merged 2 commits into
mainfrom
lilac/fmt
Jun 5, 2025
Merged

chore: simplify prettier config#528
aslilac merged 2 commits into
mainfrom
lilac/fmt

Conversation

@aslilac

@aslilac aslilac commented Jun 3, 2025

Copy link
Copy Markdown
Member

Pretty much all of the settings in the prettier config here are either in conflict with coder/coder, or are just the defaults now. We can just remove the whole file to get closer to matching coder/coder.

@aslilac

aslilac commented Jun 3, 2025

Copy link
Copy Markdown
Member Author

I would also like to switch this repository to tabs, but that will be a much bigger diff.

I decided I might as well make the diff bigger. Go big or go home.

@matifali

matifali commented Jun 4, 2025

Copy link
Copy Markdown
Member

We don't use prettier in coder/coder anymore. Maybe move to biome here too.

@aslilac

aslilac commented Jun 5, 2025

Copy link
Copy Markdown
Member Author

I think the editor integration for prettier is pretty universally better. in a smaller repo like this where it's fast enough I actually think it's better to stick with prettier.

@aslilac aslilac reopened this Jun 5, 2025
@aslilac aslilac requested review from bcpeinhardt and matifali June 5, 2025 17:55
@aslilac aslilac merged commit f785902 into main Jun 5, 2025
2 of 4 checks passed
@aslilac aslilac deleted the lilac/fmt branch June 5, 2025 20:15
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.

3 participants