Faraday: RuboCopクエストの問題

作成日 2019年02月20日 · 5コメント · ソース: lostisland/faraday

基本情報

RuboCopを使用してコードベース全体をリントしています。

これは、開始時の.rubocop_todo.yml内のすべてのルールのリストです。チェックマークは、「これは修正され、マスターにマージされる」ことを意味します。

これらのいずれかを修正するPRを作成する場合は、RuboCopの--onlyオプションを使用して、1種類の修正に集中してください。このようなコードをレビューするとき、圧倒されるのはとても簡単です。（それより少し多く修正しても、大したことはありません。これは、変更をレビュー可能に保つことです。）

# 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] Layout / AlignHash＃863
[x] Layout / AlignParameters＃867
[x]レイアウト/ BlockEndNewline
[x]レイアウト/ケースインデンテーション＃864
[x]レイアウト/ CommentIndentation＃864
[x]レイアウト/ドット位置＃880
[x]レイアウト/ ElseAlignment＃871
[x] Layout / EmptyLineAfterGuardClause＃860
[x] Layout / EmptyLineBetweenDefs＃860
[x]レイアウト/ EmptyLines＃860
[x] Layout / EmptyLinesAroundAccessModifier＃881
[x] Layout / EmptyLinesAroundClassBody＃882
[x] Layout / EndAlignment＃887
[x]レイアウト/ ExtraSpacing＃884
[x]レイアウト/ IndentArray＃890
[x]レイアウト/ IndentHash＃890
[x]レイアウト/インデンテーション幅＃864
[x] Layout / LeadingCommentSpace＃860
[x]レイアウト/ MultilineBlockLayout
[x] Layout / MultilineMethodCallIndentation＃864
[x]レイアウト/ MultilineOperationIndentation＃864
[x] Layout / RescueEnsureAlignment＃890
[x]レイアウト/ SpaceAfterComma＃884
[x] Layout / SpaceAroundEqualsInParameterDefault＃884
[x]レイアウト/ SpaceAroundOperators＃884
[x] Layout / SpaceBeforeBlockBraces＃884
[x] Layout / SpaceInsideArrayLiteralBrackets＃884
[x] Layout / SpaceInsideBlockBraces＃884
[x] Layout / SpaceInsideHashLiteralBraces＃884
[x] Layout / SpaceInsideParens＃884
[x] Layout / SpaceInsidePercentLiteralDelimiters＃884
[x] Layout / TrailingBlankLines＃872
[x]レイアウト/ TrailingWhitespace
[x] Lint / AssignmentInCondition＃874
[x] Lint / HandleExceptions＃896
[x] Lint / ReturnInVoidContext＃896
[x] Lint / StringConversionInInterpolation＃891
[x] Lint / UnneededRequireStatement＃896
[x] Lint / UnusedBlockArgument＃896
[x] Lint / UnusedMethodArgument＃896
[x] Lint / UselessAssignment＃891
[x] Lint / Void＃896
[x] _無期限に延期_〜Metrics / AbcSize〜
[x]メトリクス/ブロック長＃883＃927
[x] _無期限に延期_〜Metrics / ClassLength＃944〜
[x] _無期限に延期_〜メトリクス/循環的複雑度〜
[x] _無期限に延期_〜Metrics / MethodLength〜
[x] _無期限に延期_〜Metrics / ModuleLength〜
[x] _無期限に延期_〜Metrics / PerceivedComplexity〜
[x]ネーミング/定数名＃897
[x]ネーミング/ MemoizedInstanceVariableName＃891
[x]ネーミング/述語名＃897
[x]ネーミング/ UncommunicativeMethodParamName＃897
[x]パフォーマンス/冗長ブロックコール＃897
[x]パフォーマンス/ StringReplacement＃897
[x] Style / AccessModifierDeclarations＃897
[x]スタイル/エイリアス＃897
[x]スタイル/ AndOr＃897
[x]スタイル/ BlockDelimiters＃906
[x] Style / BracesAroundHashParameters＃907
[x] Style / CaseEquality＃902
[x] Style / ClassAndModuleChildren＃902
[x]スタイル/クラスチェック＃902
[x] Style / ClassVars＃933
[x] Style / ColonMethodCall＃894
[x]スタイル/コメント注釈＃876
[x] Style / ConditionalAssignment＃902
[x]スタイル/ドキュメント
[x]スタイル/ 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]スタイル/ GuardClause＃931
[x]スタイル/ハッシュ構文＃878
[x]スタイル/ IfInsideElse＃915
[x] Style / IfUnlessModifier＃926
[x]スタイル/ラムダ＃901
[x]スタイル/メソッドMissingSuper＃928
[x] Style / MissingRespondToMissing＃930
[x]スタイル/モジュール機能＃925
[x] Style / MultilineIfModifier＃924
[x]スタイル/ MultilineTernaryOperator＃913
[x]スタイル/複数比較＃923
[x] Style / MutableConstant＃925
[x]スタイル/ NegatedIf＃901
[x] Style / NestedParenthesizedCalls＃901
[x]スタイル/次へ＃901
[x]スタイル/ NilComparison＃921
[x]スタイル/ Not＃901
[x]スタイル/数値述語＃921
[x] Style / ParallelAssignment＃920
[x] Style / ParenthesesAroundCondition＃920
[x] Style / PercentLiteralDelimiters＃905
[x]スタイル/ PerlBackrefs＃920
[x] Style / PreferredHashMethods＃920
[x]スタイル/手順＃920
[x] Style / RaiseArgs＃910
[x]スタイル/冗長例外＃895
[x]スタイル/冗長Parentheses＃895
[x]スタイル/冗長リターン＃895
[x]スタイル/冗長セルフ＃895
[x]スタイル/ RegexpLiteral＃910
[x] Style / RescueStandardError＃910
[x]スタイル/ SafeNavigation＃909
[x]スタイル/セミコロン＃908
[x] Style / SingleLineMethods＃909
[x] Style / SpecialGlobalVars＃909
[x]スタイル/ StderrPuts＃908
[x]スタイル/文字列リテラル＃874
[x] Style / StringLiteralsInInterpolation＃909
[x]スタイル/構造継承＃904
[x] Style / SymbolArray＃898
[x] Style / SymbolProc＃898
[x]スタイル/ TernaryParentheses＃898
[x] Style / TrailingCommaInArrayLiteral＃859
[x] Style / TrailingCommaInHashLiteral＃859
[x]スタイル/ TrivialAccessors＃898
[x]スタイル/ UnlessElse＃898
[x]スタイル/ WordArray＃893
[x]スタイル/ヨーダコンディション＃898
[x] Style / ZeroLengthPredicate＃858
[x]メトリック/ LineLength＃938

help wanted

ソース

olleolleolle

👍1

最も参考になるコメント

みんな、ありがとう！ 🚀

olleolleolle 2019年04月08日

🎉3

全てのコメント5件

このコメントを削除します。私の以前の主張は間違っていました。将来の開発者は、違反をチェックするときに.rubocop_todo.ymlファイルに注意する必要があります。

jherdman 2019年02月25日

現在のマスター参照には、CIによってキャッチされないいくつかのrubocop違反があります。これらの問題は、 https：//github.com/lostisland/faraday/pull/870およびhttps://github.com/lostisland/faraday/pull/871で修正されてい

また、PRに[RuboCop] fix Layout/TrailingBlankLinesような名前を付けると、上記のチェックリストに追加するのがはるかに簡単になります。

technoweenie 2019年02月25日

唯一の違反者はscript/generate_certsため、 Style/GlobalVarsをチェックしました。古い統合テストが完全に移植されると、統合スイートの古いスクリプトを実行できるようになり、この問題は魔法のように修正されます。他の場所に追加された他のグローバル変数は、引き続きrubocop違反としてフラグを立てる必要があります。

technoweenie 2019年03月05日

👍2

Style/Globalvarsを修正する前にこの投稿を見たことがありません

Insti 2019年03月10日