Estamos lintando toda a base de código usando RuboCop.
Esta é uma lista de todas as regras em .rubocop_todo.yml
quando começamos. As marcas de seleção significam "isso foi corrigido e mesclado com o mestre".
Se você criar um PR para corrigir um desses, use a opção --only
no RuboCop, para concentrar o esforço em 1 tipo de correção. É tão fácil ficar sobrecarregado ao revisar um código como este. (Se você corrigir um pouco mais do que isso, não é nada demais. O objetivo é manter as alterações revisáveis.)
# edit .rubocop.yml, commenting out the first line -
# this removes the "ignore the TODOs" setting
# Auto-correct with only 1 rule
rubocop -a --only Name/OfTheRuleYouAreFixing --auto-correct
# Re-generate the config file
rubocop --auto-gen-config
# Revert line 3 in .rubocop_todo.yml
# This will avoid PR conflicts
# Inspect to see if it looks OK
git diff
# Re-set the TODO list
git checkout .rubocop.yml
# Add and commit the change
git add .
git commit -m"chore: RuboCop lint Name/OfTheRuleYouAreFixing"
Removendo este comentário. Minha afirmação anterior estava errada. Os desenvolvedores futuros devem observar o arquivo .rubocop_todo.yml ao verificar as violações.
O atual mestre ref tem algumas violações do rubocop que não foram detectadas pelo CI. Esses problemas foram corrigidos em https://github.com/lostisland/faraday/pull/870 e https://github.com/lostisland/faraday/pull/871.
Além disso, se os PRs forem nomeados como [RuboCop] fix Layout/TrailingBlankLines
, será muito mais fácil adicioná-los à lista de verificação acima.
Eu marquei Style/GlobalVars
porque o único infrator é script/generate_certs
. Uma vez que os testes de integração antigos foram totalmente portados, os scripts antigos para o pacote de integração podem ir, e esta ofensa será corrigida magicamente. Quaisquer outras variáveis globais adicionadas em outro lugar devem continuar a ser sinalizadas como ofensas rubocop.
Eu não tinha visto esta postagem antes de consertar o Style/Globalvars
: man_shrugging:
Obrigado a todos! 🚀
Comentários muito úteis
Obrigado a todos! 🚀