developmentseed / devseed-standard

1 stars 1 forks source link

Integration with prettier #2

Open AliceR opened 3 years ago

AliceR commented 3 years ago

My project (based on developmentseed/cookiecutter-browserify-react-mapboxgl) is using devseed-standard for linting and prettier for formatting. When I change the formatting preferences in .prettierrc, devseed-standard does not adhere to the same rules, but throws linting errors.

I wonder if we should integrate prettier directly into devseed-standard, shifting all formatting responsibility to prettier and use eslint for linting only. This is the recommended approach in the documentation as well, see https://prettier.io/docs/en/integrating-with-linters.html.

@sethvincent Would you agree to this approach?

sethvincent commented 3 years ago

@AliceR i've thought about integrating prettier into this repo, too. it would make things a lot easier to set up and i would generally be for it.

potential alternate approach: lately i've been trying out rome a little more in my side project tinkering time and i'm really starting to like it. i'd be tempted to stop dealing with eslint + prettier setups and just use rome for both linting and formatting.

AliceR commented 3 years ago

🚚 Fetching packages... 🔗 Linking dependencies...

✨ Done in 6.74s.

Welcome to Rome! Let's see how that goes...