نحن نقوم بفحص قاعدة الكود بأكملها باستخدام RuboCop.
هذه قائمة بجميع القواعد في .rubocop_todo.yml
عندما بدأنا. تعني علامات الاختيار "هذا تم إصلاحه ودمجه في الأساس".
إذا قمت بإنشاء علاقات عامة لإصلاح أحدها ، فاستخدم الخيار --only
في RuboCop ، لتركيز الجهد على نوع واحد من الإصلاح. من السهل جدًا الشعور بالضيق عند مراجعة رمز مثل هذا. (إذا قمت بإصلاح أكثر من ذلك بقليل ، فلا مشكلة كبيرة. هذا كله يتعلق بإبقاء التغييرات قابلة للمراجعة.)
# 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"
إزالة هذا التعليق. تأكيدي السابق كان خاطئا. يجب على المطورين المستقبليين ملاحظة ملف .rubocop_todo.yml عند التحقق من الانتهاكات.
يحتوي المرجع الرئيسي الحالي على بعض انتهاكات rubocop التي لم يتم القبض عليها بواسطة CI. تم إصلاح هذه المشكلات في https://github.com/lostisland/faraday/pull/870 و https://github.com/lostisland/faraday/pull/871.
أيضًا ، إذا تم تسمية العلاقات العامة بشيء مثل [RuboCop] fix Layout/TrailingBlankLines
، فسيكون من الأسهل إضافتهم إلى قائمة التحقق أعلاه.
لقد تحققت من Style/GlobalVars
لأن الجاني الوحيد هو script/generate_certs
. بمجرد أن يتم تنفيذ اختبارات التكامل القديمة بالكامل ، يمكن أن تذهب البرامج النصية القديمة لمجموعة التكامل ، وسيتم إصلاح هذه المشكلة بطريقة سحرية. يجب الاستمرار في تمييز أي متغيرات عالمية أخرى تمت إضافتها في أي مكان آخر على أنها جرائم روبوكوب.
لم أشاهد هذا المنشور قبل أن أصلح Style/Globalvars
: man_shrugging:
شكرا لكم جميعا! 🚀
التعليق الأكثر فائدة
شكرا لكم جميعا! 🚀