Replies: 1 comment
-
That's really true. And I also like I've updated the lint rules. It will be introduced into the core soon! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Several PRs have recently added changes as refactorings that apply the following rules
https://eslint.org/docs/latest/rules/curly#multi
If I may say my preference, I like
#all
, but leaving that aside, the "brace style" of "if statement" is a problem in modern JavaScript that should be solved with a linter, and a review in the PR I don't think it is a problem that should be solved at the cost of a PR review.If the project decides on a rule (
#multi
or#all
or …), it should be left to a linter, and if not, I think PR should be prohibited from changing "if statements".Beta Was this translation helpful? Give feedback.
All reactions