search

pagarme / javascript-style-guide

:art: Javascript styleguide followed by us here at Pagar.me
MIT License
50 stars 8 forks source link

Disallow multiple empty lines #27

Closed matheusvellone closed 6 years ago

matheusvellone commented 6 years ago

I think disallowing multiple empty lines very useful to keep the code visually consistent. This PR adds the following rule to classify 2+ empty lines as an error.

matheusvellone commented 6 years ago

Exemplos e breve descrição no README adicionados :wink:

mccraveiro commented 6 years ago

@MatheusVellone O airbnb que usamos como base já possui essa regra, porque colocar explicitamente?

matheusvellone commented 6 years ago

Se possui a regra, ela não funciona. Vou procurar ela nas configs do airbnb pra conferir

Mas fiz um teste: coloquei 2 linhas vazias e não estava acusando erro. Coloquei a regra explicita no .eslintrc.js e pegou

matheusvellone commented 6 years ago

@mccraveiro achei a regra do airbnb

Ele define da seguinte forma:

https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js#L305

'no-multiple-empty-lines': ['error', { max: 2, maxEOF: 0 }],

Pessoalmente prefiro que o max seja 1 (tbm no EOF). Acho que o máximo sendo 2 linhas de separação pode ser "nocivo" pro código por aceitar 2 formas (1 ou 2 espaços) e dessa forma não estabelecendo um padrão.