quero-edu / guidelines

📘 The definitive Quero Education code-style guide.
MIT License
22 stars 2 forks source link

replace eslint base rules in favor of TS rules #60

Closed mateusppereira closed 3 years ago

toledompm commented 3 years ago

Algumas regras conflitantes que desligamos não são enforçadas pelo config base, não sei se faz sentido aplicarmos elas apenas para os projetos typescript (quando coloquei todas essas regras no gateway teve >8000 erros :medo:). Essas são as regras:

default-param-last
dot-notation
func-call-spacing
init-declarations
lines-between-class-members
no-array-constructor
no-dupe-class-members
no-empty-function
no-extra-parens
no-implied-eval
no-loop-func
no-loss-of-precision
no-magic-numbers
no-redeclare
no-shadow
no-unused-expressions
no-use-before-define
no-useless-constructor
require-await
return-await
space-before-function-paren

Algumas regras também tem opções especiais no config base que precisamos replicar:

'indent': ['error', 2, { 'SwitchCase': 1 }]
'comma-dangle': ['error', 'always-multiline']
'keyword-spacing': [ 'error', { 'after': true, 'before': true }]
'no-unused-vars': ['error', {'vars': 'all', 'args': 'all', 'argsIgnorePattern': '^_', 'ignoreRestSiblings': true}]
'object-curly-spacing': ['error', 'always']
'padding-line-between-statements': [
      'error',
      { 'blankLine': 'always', 'prev': '*', 'next': ['block-like', 'multiline-expression'] },
      { 'blankLine': 'always', 'prev': ['block-like', 'multiline-expression'], 'next': '*' },
 ]
 'quotes': ['error', 'single'],
toledompm commented 3 years ago

@vitebo

Só uma dúvida, você conseguiu testar isso aqui em algum repo? Pq se não, eu consigo publicar uma release alpha desse cara tbm, ai fica mais facil de validar, mas se você achar que ta safe tbm, por mim pode mandar bala!

ctrl + c -> ctrl + v nas regras no stock-gateway :v