Skip to content

Conversation

EdvinasDaugirdas
Copy link

@EdvinasDaugirdas EdvinasDaugirdas commented Sep 4, 2025

What?

Update the documentation to replace eslint+prettier with eslint in the toolchain value.

Why?

eslint+prettier is not a valid toolchain value. Valid options are eslint and biome.

Screenshots

Screenshot 2025-09-04 at 11 23 25

@EdvinasDaugirdas
Copy link
Author

EdvinasDaugirdas commented Sep 4, 2025

Wanted to add some additional context and thoughts. Feel free to ignore this comment if you think it's irrelevant.

The previous value, eslint+prettier, was more explicit, but I can understand why the name was changed.

With that being said, it's not obvious that by selecting eslint, prettier will be set up too. I do understand that it is mentioned in the Toolchain section below, but that could easily be missed. Perhaps a mention could be brought up a bit earlier on, next to the place where the valid values are mentioned? Something like:

--toolchain: Specify your toolchain solution for formatting/linting (biome, eslint). Note that by selecting eslint prettier will be configured too.

The same goes for the project scaffolding flow in the CLI. The result would be more obvious if instead of ESLint say, f.e., ESLint + Prettier option would be provided.

Screenshot 2025-09-04 at 11 33 30

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.

1 participant