Nous lintons toute la base de code à l'aide de RuboCop.
Ceci est une liste de toutes les règles du .rubocop_todo.yml
lorsque nous avons commencé. Les coches signifient « ceci est corrigé et fusionné avec le maître ».
Si vous créez un PR pour corriger l'un d'entre eux, utilisez l'option --only
dans RuboCop, pour concentrer l'effort sur 1 type de correction. Il est si facile de se laisser submerger lors de la révision de code comme celui-ci. (Si vous corrigez un peu plus que cela, pas de problème. Il s'agit de garder les modifications révisables.)
# 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"
Suppression de ce commentaire. Mon affirmation précédente était fausse. Les futurs développeurs doivent noter ce fichier .rubocop_todo.yml lors de la vérification des violations.
La référence principale actuelle a des violations de rubocop qui ne sont pas détectées par CI. Ces problèmes sont résolus dans https://github.com/lostisland/faraday/pull/870 et https://github.com/lostisland/faraday/pull/871.
De plus, si les PR sont nommés quelque chose comme [RuboCop] fix Layout/TrailingBlankLines
, alors ils seront beaucoup plus faciles à ajouter à la liste de contrôle ci-dessus.
J'ai coché Style/GlobalVars
parce que le seul contrevenant est script/generate_certs
. Une fois les anciens tests d'intégration entièrement portés, les anciens scripts de la suite d'intégration peuvent disparaître et cette infraction sera corrigée comme par magie. Toutes les autres variables globales ajoutées ailleurs devraient continuer à être signalées comme des infractions rubocop.
Je n'avais pas vu ce post avant de corriger le Style/Globalvars
:man_shruging :
Merci tout le monde! ??
Commentaire le plus utile
Merci tout le monde! ??