Code style about webpack-blocks HOT 6 CLOSED

I understand that eslint handles a whole class of things that prettier does not, but I kind of like to talk about those things during code review anyway.

That is true for code style, but the important "whole class of things that prettier does not handle" is stuff like checking for unused variables, broken references, etc. That's what prettier cannot provide. I say it should not be the reviewer's responsibility either, since it can be reliably checked by software 😉

from webpack-blocks.

Could you please link to those places with different code styles?

Using prettier has been a topic for quite some time. I don't see any reason against, there was just no time yet to do that.

How would the Airbnb ruleset change anything? Because it's stricter?

from webpack-blocks.

@andywer

Using prettier has been a topic for quite some time. I don't see any reason against, there was just no time yet to do that.

Setting up Prettier and ESLint is really easy, I've even created an ESLint preset that automatically configures Prettier and the Airbnb preset.

How would the Airbnb ruleset change anything? Because it's stricter?

No. It's just my preference.

from webpack-blocks.

I prefer husky + prettier and no eslint. I understand that eslint handles a whole class of things that prettier does not, but I kind of like to talk about those things during code review anyway.

from webpack-blocks.

Is this issue still relevant, now that we have prettier in the v2.0 branch?

from webpack-blocks.

We've got both Prettier and ESLint. Now we can add more ESLint rules over time.

from webpack-blocks.