Estamos uniendo todo el código base usando RuboCop.
Esta es una lista de todas las reglas en el .rubocop_todo.yml
cuando comenzamos. Las marcas de verificación significan "esto se corrigió y se fusionó con el maestro".
Si crea un PR para corregir uno de estos, use la opción --only
en RuboCop, para centrar el esfuerzo en 1 tipo de corrección. Es muy fácil sentirse abrumado al revisar un código como este. (Si corrige un poco más que eso, no es problema. Se trata de mantener los cambios revisables).
# 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"
Eliminando este comentario. Mi afirmación anterior estaba equivocada. Los futuros desarrolladores deben tener en cuenta que el archivo .rubocop_todo.yml al verificar si hay violaciones.
La referencia maestra actual tiene algunas infracciones de Rubocop que CI no detecta. Esos problemas se solucionan en https://github.com/lostisland/faraday/pull/870 y https://github.com/lostisland/faraday/pull/871.
Además, si los RP se denominan algo así como [RuboCop] fix Layout/TrailingBlankLines
, será mucho más fácil agregarlos a la lista de verificación anterior.
Marqué Style/GlobalVars
porque el único infractor es script/generate_certs
. Una vez que las antiguas pruebas de integración se hayan portado por completo, los antiguos scripts para la suite de integración pueden desaparecer, y esta infracción se solucionará mágicamente. Cualquier otra variable global agregada en otro lugar debe seguir marcada como delitos rubocop.
No había visto esta publicación antes de arreglar el Style/Globalvars
: man_shrugging:
¡Gracias a todos! 🚀
Comentario más útil
¡Gracias a todos! 🚀