Faraday: Masalah Quest RuboCop

Dibuat pada 20 Feb 2019  ·  5Komentar  ·  Sumber: lostisland/faraday

Info dasar

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"

Alur kerja

Deskripsi masalah

  • [x] Bundler/Permata yang Dipesan #856
  • [x] Layout/AccessModifierIndentation #862
  • [x] Tata Letak/AlignHash #863
  • [x] Layout/AlignParameters #867
  • [x] Tata Letak/BlockEndNewline
  • [x] Layout/CaseIndentation #864
  • [x] Layout/CommentIndentation #864
  • [x] Tata Letak/Posisi Titik #880
  • [x] Tata Letak/ElseAlignment #871
  • [x] Tata Letak/EmptyLineAfterGuardClause #860
  • [x] Tata Letak/EmptyLineBetweenDefs #860
  • [x] Tata Letak/EmptyLines #860
  • [x] Layout/EmptyLinesAroundAccessModifier #881
  • [x] Layout/EmptyLinesAroundClassBody #882
  • [x] Tata Letak/EndAlignment #887
  • [x] Layout/ExtraSpacing #884
  • [x] Tata Letak/IndentArray #890
  • [x] Tata Letak/IndentHash #890
  • [x] Layout/Lebar Indentasi #864
  • [x] Tata Letak/LeadingCommentSpace #860
  • [x] Tata Letak/MultilineBlockLayout
  • [x] Layout/MultilineMethodCallIndentation #864
  • [x] Layout/MultilineOperationIndentation #864
  • [x] Tata Letak/RescueEnsureAlignment #890
  • [x] Tata Letak/SpaceAfterComma #884
  • [x] Layout/SpaceAroundEqualsInParameterDefault #884
  • [x] Layout/SpaceAroundOperators #884
  • [x] Layout/SpaceBeforeBlockBraces #884
  • [x] Layout/SpaceInsideArrayLiteralBrackets #884
  • [x] Layout/SpaceInsideBlockBraces #884
  • [x] Layout/SpaceInsideHashLiteralBraces #884
  • [x] Tata Letak/SpaceInsideParens #884
  • [x] Layout/SpaceInsidePercentLiteralDelimiters #884
  • [x] Tata Letak/TrailingBlankLines #872
  • [x] Tata Letak/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] Serat/Void #896
  • [x] _Ditunda tanpa batas waktu_ ~Metrics/AbcSize~
  • [x] Metrik/BlockLength #883 #927
  • [x] _Ditunda tanpa batas waktu_ ~Metrics/ClassLength #944~
  • [x] _Ditunda tanpa batas waktu_ ~Metrics/CyclomaticComplexity~
  • [x] _Ditunda tanpa batas waktu_ ~Metrics/MethodLength~
  • [x] _Ditunda tanpa batas waktu_ ~Metrics/ModuleLength~
  • [x] _Ditunda tanpa batas waktu_ ~Metrik/PerceivedComplexity~
  • [x] Penamaan/ConstantName #897
  • [x] Penamaan/MemoizedInstanceVariableName #891
  • [x] Penamaan/Nama Predikat #897
  • [x] Naming/UncommunicativeMethodParamName #897
  • [x] Kinerja/RedundantBlockCall #897
  • [x] Performa/Penggantian String #897
  • [x] Style/AccessModifierDeclarations #897
  • [x] Gaya/Alias ​​#897
  • [x] Gaya/DanAtau #897
  • [x] Style/BlockDelimiters #906
  • [x] Style/BraceAroundHashParameters #907
  • [x] Gaya/Kasus Kesetaraan #902
  • [x] Style/ClassAndModuleChildren #902
  • [x] Gaya/Periksa Kelas #902
  • [x] Style/ClassVars #933
  • [x] Style/ColonMethodCall #894
  • [x] Style/CommentAnnotation #876
  • [x] Tugas Gaya/Kondisi #902
  • [x] Gaya/Dokumentasi
  • [x] Gaya/Negasi Ganda #922
  • [x] Style/EachWithObject #901
  • [x] Gaya/Metode Kosong #912
  • [x] Style/ExpandPathArguments #901
  • [x] Gaya/FormatString #901
  • [x] Gaya/FormatStringToken #914
  • [x] Style/FrozenStringLiteralComment #868
  • [x] Gaya/GlobalVars #943
  • [x] Style/GuardClause #931
  • [x] Gaya/HashSyntax #878
  • [x] Style/IfInsideElse #915
  • [x] Style/IfUnlessModifier #926
  • [x] Gaya/Lambda #901
  • [x] Gaya/Metode HilangSuper #928
  • [x] Style/MissingRespondToMissing #930
  • [x] Gaya/ModulFungsi #925
  • [x] Style/MultilineIfModifier #924
  • [x] Style/MultilineTernaryOperator #913
  • [x] Gaya/Perbandingan Ganda #923
  • [x] Gaya/MutableConstant #925
  • [x] Style/NegatedIf #901
  • [x] Style/NestedParenthesizedCalls #901
  • [x] Gaya/Berikutnya #901
  • [x] Gaya/NihilPerbandingan #921
  • [x] Gaya/Bukan #901
  • [x] Predikat Gaya/Numerik #921
  • [x] Style/ParallelAssignment #920
  • [x] Gaya/KurungAroundCondition #920
  • [x] Style/PercentLiteralDelimiters #905
  • [x] Style/PerlBackrefs #920
  • [x] Style/PreferredHashMethods #920
  • [x] Gaya/Proc #920
  • [x] Style/RaiseArgs #910
  • [x] Style/RedundantException #895
  • [x] Gaya/Kurung Berlebihan #895
  • [x] Gaya/Pengembalian Berlebihan #895
  • [x] Gaya/RedundantSelf #895
  • [x] Gaya/RegexpLiteral #910
  • [x] Style/RescueStandardError #910
  • [x] Gaya/Navigasi Aman #909
  • [x] Gaya/Titik Koma #908
  • [x] Gaya/SingleLineMethods #909
  • [x] Style/SpecialGlobalVars #909
  • [x] Gaya/StderrPuts #908
  • [x] Style/StringLiterals #874
  • [x] Style/StringLiteralsInInterpolation #909
  • [x] Style/StructInheritance #904
  • [x] Style/SymbolArray #898
  • [x] Style/SymbolProc #898
  • [x] Style/TernaryKurung #898
  • [x] Style/TrailingCommaInArrayLiteral #859
  • [x] Style/TrailingCommaInHashLiteral #859
  • [x] Style/SepeleAccessors #898
  • [x] Gaya/Kecuali Lain #898
  • [x] Style/WordArray #893
  • [x] Style/YodaCondition #898
  • [x] Predikat Style/ZeroLength #858
  • [x] Metrik/Panjang Garis #938
help wanted

Komentar yang paling membantu

Terimakasih semuanya! 🚀

Semua 5 komentar

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! 🚀

Apakah halaman ini membantu?
0 / 5 - 0 peringkat