Kami linting seluruh basis kode menggunakan RuboCop.
Ini adalah daftar semua aturan di .rubocop_todo.yml
saat kita memulai. Tanda centang berarti "ini sudah diperbaiki dan digabungkan menjadi master".
Jika Anda membuat PR untuk memperbaiki salah satunya, gunakan opsi --only
di RuboCop, untuk memfokuskan upaya pada 1 jenis perbaikan. Sangat mudah untuk kewalahan ketika meninjau kode seperti ini. (Jika Anda memperbaiki sedikit lebih dari itu, tidak masalah. Ini semua tentang menjaga agar perubahan tetap dapat ditinjau.)
# 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"
Menghapus komentar ini. Pernyataan saya sebelumnya salah. Pengembang masa depan harus memperhatikan bahwa file .rubocop_todo.yml saat memeriksa pelanggaran.
Master ref saat ini memiliki beberapa pelanggaran rubocop yang tidak ditangkap oleh CI. Masalah tersebut diperbaiki di https://github.com/lostisland/faraday/pull/870 dan https://github.com/lostisland/faraday/pull/871.
Juga, jika PR diberi nama seperti [RuboCop] fix Layout/TrailingBlankLines
, maka mereka akan lebih mudah ditambahkan ke daftar periksa di atas.
Saya memeriksa Style/GlobalVars
karena satu-satunya pelanggar adalah script/generate_certs
. Setelah tes integrasi lama telah sepenuhnya di-porting, skrip lama untuk suite integrasi dapat digunakan, dan pelanggaran ini akan diperbaiki secara ajaib. Setiap vars global lainnya yang ditambahkan di tempat lain harus terus ditandai sebagai pelanggaran rubocop.
Saya belum pernah melihat posting ini sebelum saya memperbaiki Style/Globalvars
:man_shrugging:
Terimakasih semuanya! 🚀
Komentar yang paling membantu
Terimakasih semuanya! 🚀