search

dimdenGD / ultimate-express

The Ultimate Express. Fastest http server with full Express compatibility, based on µWebSockets.
Apache License 2.0
519 stars 15 forks source link

feat: eslint, cspell, prettier #47

Closed nigrosimone closed 2 weeks ago

nigrosimone commented 2 weeks ago

add eslint, cspell, prettier

npm run test

also check linting, formatting and spelling

This PR cause a massive changes due to follow linting, formatting and spelling best practice

dimdenGD commented 2 weeks ago

Sorry, I'm not interested in having these things in the project

nigrosimone commented 2 weeks ago

Why not? eslint, cspell and prettier are must to have in large project with multiple contributors. ESLint enables developers to discover problems with their code without actually executing their code. Prettier enforces a consistent code style across your entire codebase.

dimdenGD commented 2 weeks ago

I find linters and prettier to be distrupting my workflow and code style a lot. And spell checkers to have false positives often.

nigrosimone commented 2 weeks ago

Ok, I undestand. I have also added extensions for VSCode that help and a "fix" command for autofix. Try It. It's not invasive and help a lot.