Faraday: قضية مهمة RuboCop

تم إنشاؤها على ٢٠ فبراير ٢٠١٩  ·  5تعليقات  ·  مصدر: lostisland/faraday

معلومات أساسية

نحن نقوم بفحص قاعدة الكود بأكملها باستخدام 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"

سير العمل

وصف المشكلة

  • [x] Bundler / OrderedGems # 856
  • [x] Layout / AccessModifierIndentation # 862
  • [x] تخطيط / AlignHash # 863
  • [x] Layout / AlignParameters # 867
  • [x] تخطيط / BlockEndNewline
  • [x] التخطيط / CaseIndentation # 864
  • [x] تنسيق / مسافة بادئة # 864
  • [x] التخطيط / DotPosition # 880
  • [x] التخطيط / ElseAlignment # 871
  • [x] تخطيط / EmptyLineAfterGuardClause # 860
  • [x] تخطيط / EmptyLineBetweenDefs # 860
  • [x] تخطيط / EmptyLines # 860
  • [x] Layout / EmptyLinesAroundAccessModifier # 881
  • [x] تخطيط / EmptyLinesAroundClassBody # 882
  • [x] التخطيط / EndAlignment # 887
  • [x] تخطيط / مسافة إضافية # 884
  • [x] Layout / IndentArray # 890
  • [x] تخطيط / تجزئة مسافة بادئة # 890
  • [x] التخطيط / عرض المسافة البادئة # 864
  • [x] تخطيط / LeadingCommentSpace # 860
  • [x] تخطيط / MultilineBlockLayout
  • [x] Layout / MultilineMethodCallIndentation # 864
  • [x] تخطيط / عملية متعددة الخطوط المسافة البادئة # 864
  • [x] Layout / RescueEnsureAlignment # 890
  • [x] تخطيط / SpaceAfterComma # 884
  • [x] التخطيط / SpaceAroundEqualsInParameterDefault # 884
  • [x] Layout / SpaceAroundOperators # 884
  • [x] Layout / SpaceBeforeBlockBraces # 884
  • [x] Layout / SpaceInsideArrayLiteralBrackets # 884
  • [x] التخطيط / SpaceInsideBlockBraces # 884
  • [x] Layout / SpaceInsideHashLiteralBraces # 884
  • [x] تخطيط / SpaceInsideParens # 884
  • [x] Layout / SpaceInsidePercentLiteralDelimiters # 884
  • [x] التخطيط / TrailingBlankLines # 872
  • [x] تخطيط / مسافة بيضاء زائدة
  • [x] Lint / AssignmentInCondition # 874
  • [x] Lint / HandleExceptions # 896
  • [x] Lint / ReturnInVoidContext # 896
  • [x] لينت / StringConversionInterpolation # 891
  • [x] Lint / UnneededRequireStatement # 896
  • [x] Lint / UnusedBlockArgument # 896
  • [x] Lint / UnusedMethodArgument # 896
  • [x] لينت / UselessAssignment # 891
  • [x] لينت / باطل # 896
  • [x] _ تم تأجيله إلى أجل غير مسمى_ ~ المقاييس / AbcSize ~
  • [x] المقاييس / طول الكتلة # 883 # 927
  • [x] _ تم تأجيله إلى أجل غير مسمى _ ​​المقاييس / ClassLength # 944 ~
  • [x] _ تم تأجيله إلى أجل غير مسمى _ ​​المقاييس / التعقيد السيكلوماتي ~
  • [x] _ تم تأجيله إلى أجل غير مسمى _ ​​المقاييس / طول الطريقة ~
  • [x] _ تم تأجيله إلى أجل غير مسمى _ ​​المقاييس / طول الوحدة ~
  • [x] _ تم تأجيله إلى أجل غير مسمى _ ​​المقاييس / التعقيد المدرك ~
  • [x] تسمية / اسم ثابت # 897
  • [x] تسمية / MemoizedInstanceVariableName # 891
  • [x] Naming / PredicateName # 897
  • [x] تسمية / UncommunicativeMethodParamName # 897
  • [x] الأداء / RedundantBlockCall # 897
  • [x] الأداء / StringReplacement # 897
  • [x] Style / AccessModifierDeclarations # 897
  • [x] النمط / الاسم المستعار # 897
  • [x] النمط / AndOr # 897
  • [x] Style / BlockDelimiters # 906
  • [x] Style / BracesAroundHashParameters # 907
  • [x] Style / CaseEquality # 902
  • [x] Style / ClassAndModuleChildren # 902
  • [x] Style / ClassCheck # 902
  • [x] Style / ClassVars # 933
  • [x] Style / ColonMethodCall # 894
  • [x] Style / CommentAnnotation # 876
  • [x] Style / ConditionalAssignment # 902
  • [x] الأسلوب / التوثيق
  • [x] Style / DoubleNegation # 922
  • [x] Style / EachWithObject # 901
  • [x] Style / EmptyMethod # 912
  • [x] Style / ExpandPathArguments # 901
  • [x] Style / FormatString # 901
  • [x] النمط / FormatStringToken # 914
  • [x] Style / FrozenStringLiteralComment # 868
  • [x] Style / GlobalVars # 943
  • [x] Style / GuardClause # 931
  • [x] النمط / HashSyntax # 878
  • [x] النمط / IfInsideElse # 915
  • [x] النمط / IfUnlessModifier # 926
  • [x] ستايل / لامدا # 901
  • [x] Style / MethodMissingSuper # 928
  • [x] النمط / MissingRespondToMissing # 930
  • [x] النمط / الوظيفة # 925
  • [x] Style / MultilineIfModifier # 924
  • [x] النمط / مشغل متعدد الأسطر رقم 913
  • [x] النمط / المقارنة المتعددة # 923
  • [x] Style / MutableConstant # 925
  • [x] Style / NegatedIf # 901
  • [x] Style / NestedParenthesizedCalls # 901
  • [x] النمط / التالي # 901
  • [x] Style / NilComparison # 921
  • [x] النمط / ليس # 901
  • [x] النمط / NumericPredicate # 921
  • [x] النمط / ParallelAssignment # 920
  • [x] النمط / الأقواس AroundCondition # 920
  • [x] Style / PercentLiteralDelimiters # 905
  • [x] Style / PerlBackrefs # 920
  • [x] Style / PreferredHashMethods # 920
  • [x] Style / Proc # 920
  • [x] Style / RaiseArgs # 910
  • [x] Style / RedundantException # 895
  • [x] النمط / الأقواس الزائدة # 895
  • [x] النمط / العودة الزائدة عن الحاجة # 895
  • [x] Style / RedundantSelf # 895
  • [x] Style / RegexpLiteral # 910
  • [x] Style / RescueStandardError # 910
  • [x] Style / SafeNavigation # 909
  • [x] النمط / الفاصلة المنقوطة # 908
  • [x] Style / SingleLineMethods # 909
  • [x] Style / SpecialGlobalVars # 909
  • [x] النمط / StderrPuts # 908
  • [x] Style / StringLiterals # 874
  • [x] Style / StringLiteralsIn Interpolation # 909
  • [x] Style / StructInheritance # 904
  • [x] Style / SymbolArray # 898
  • [x] Style / SymbolProc # 898
  • [x] النمط / الأقواس الثلاثية # 898
  • [x] Style / TrailingCommaInArrayLiteral # 859
  • [x] Style / TrailingCommaInHashLiteral # 859
  • [x] Style / TrivialAccessors # 898
  • [x] النمط / ما لم عدا # 898
  • [x] النمط / WordArray # 893
  • [x] النمط / YodaCondition # 898
  • [x] Style / ZeroLengthPredicate # 858
  • [x] المقاييس / طول الخط # 938
help wanted

التعليق الأكثر فائدة

شكرا لكم جميعا! 🚀

ال 5 كومينتر

إزالة هذا التعليق. تأكيدي السابق كان خاطئا. يجب على المطورين المستقبليين ملاحظة ملف .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:

شكرا لكم جميعا! 🚀

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات