Faraday: RuboCop 任务问题

创建于 2019-02-20  ·  5评论  ·  资料来源: lostisland/faraday

基础信息

我们正在使用 RuboCop 对整个代码库进行 linting。

这是我们开始时.rubocop_todo.yml中所有规则的列表。 复选标记的意思是“这已修复并合并到母版”。

如果您创建 PR 来修复其中一种,请使用 RuboCop 中的--only选项,将精力集中在一种修复上。 在审查这样的代码时很容易不知所措。 (如果你修复的稍微多一点,没什么大不了的。这就是保持更改可审查性。)

# 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] 布局/AccessModifierIndentation #862
  • [x] 布局/AlignHash #863
  • [x] 布局/对齐参数 #867
  • [x] 布局/BlockEndNewline
  • [x] 布局/大小写缩进 #864
  • [x] 布局/注释缩进 #864
  • [x] 布局/点位置 #880
  • [x] 布局/ElseAlignment #871
  • [x] 布局/EmptyLineAfterGuardClause #860
  • [x] 布局/EmptyLineBetweenDefs #860
  • [x] 布局/空行 #860
  • [x] 布局/EmptyLinesAroundAccessModifier #881
  • [x] 布局/EmptyLinesAroundClassBody #882
  • [x] 布局/结束对齐 #887
  • [x] 布局/额外间距 #884
  • [x] 布局/缩进数组 #890
  • [x] 布局/缩进哈希 #890
  • [x] 布局/缩进宽度 #864
  • [x] 布局/LeadingCommentSpace #860
  • [x] 布局/多行块布局
  • [x] 布局/多行方法调用缩进 #864
  • [x] 布局/多行操作缩进 #864
  • [x] 布局/RescueEnsureAlignment #890
  • [x] 布局/空格后逗号 #884
  • [x] 布局/SpaceAroundEqualsInParameterDefault #884
  • [x] 布局/SpaceAroundOperators #884
  • [x] 布局/SpaceBeforeBlockBraces #884
  • [x] 布局/SpaceInsideArrayLiteralBrackets #884
  • [x] 布局/SpaceInsideBlockBraces #884
  • [x] 布局/SpaceInsideHashLiteralBraces #884
  • [x] 布局/SpaceInsideParens #884
  • [x] 布局/SpaceInsidePercentLiteralDelimiters #884
  • [x] 布局/TrailingBlankLines #872
  • [x] 布局/尾随空白
  • [x] Lint/AssignmentInCondition #874
  • [x] 皮棉/处理异常 #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] 棉绒/空隙 #896
  • [x]_无限期推迟_~指标/AbcSize~
  • [x] 指标/块长度 #883 #927
  • [x]_无限期推迟_~指标/类长度#944~
  • [x]_无限期推迟_~指标/圈复杂度~
  • [x]_无限期推迟_~指标/方法长度~
  • [x]_无限期推迟_~指标/模块长度~
  • [x]_无限期推迟_~指标/感知复杂度~
  • [x] 命名/常量名 #897
  • [x] 命名/MemoizedInstanceVariableName #891
  • [x] 命名/谓词名称 #897
  • [x] 命名/UncommunicativeMethodParamName #897
  • [x] 性能/RedundantBlockCall #897
  • [x] 性能/字符串替换 #897
  • [x] 样式/访问修饰符声明 #897
  • [x] 样式/别名 #897
  • [x] 风格/和或 #897
  • [x] 样式/块分隔符 #906
  • [x] 样式/BracesAroundHashParameters #907
  • [x] 样式/大小写相等 #902
  • [x] 样式/ClassAndModuleChildren #902
  • [x] 样式/类检查 #902
  • [x] 样式/类变量 #933
  • [x] 样式/冒号方法调用 #894
  • [x] 样式/注释注释 #876
  • [x] 样式/条件分配 #902
  • [x] 样式/文档
  • [x] 样式/双重否定 #922
  • [x] 样式/EachWithObject #901
  • [x] 样式/空方法 #912
  • [x] 样式/扩展路径参数 #901
  • [x] 样式/格式字符串 #901
  • [x] 样式/FormatStringToken #914
  • [x] 样式/FrozenStringLiteralComment #868
  • [x] 样式/全局变量 #943
  • [x] 样式/GuardClause #931
  • [x] 样式/哈希语法 #878
  • [x] 样式/IfInsideElse #915
  • [x] 样式/IfUnlessModifier #926
  • [x] 样式/Lambda #901
  • [x] 样式/方法缺失超级 #928
  • [x] 样式/MissingRespondToMissing #930
  • [x] 样式/模块功能 #925
  • [x] 样式/MultilineIfModifier #924
  • [x] 样式/多行三元运算符 #913
  • [x] 样式/多重比较 #923
  • [x] 样式/可变常量 #925
  • [x] 样式/否定如果 #901
  • [x] 样式/嵌套括号调用 #901
  • [x] 样式/下一个 #901
  • [x] 样式/无比较 #921
  • [x] 样式/非 #901
  • [x] 样式/数字谓词 #921
  • [x] 样式/并行分配 #920
  • [x] 样式/括号周围条件 #920
  • [x] 样式/百分比文字分隔符 #905
  • [x] 样式/PerlBackrefs #920
  • [x] 样式/首选哈希方法 #920
  • [x] 风格/过程 #920
  • [x] 样式/RaiseArgs #910
  • [x] 样式/冗余异常 #895
  • [x] 样式/冗余括号 #895
  • [x] 样式/冗余返回 #895
  • [x] 样式/RedundantSelf #895
  • [x] 样式/RegexpLiteral #910
  • [x] 样式/救援标准错误 #910
  • [x] 样式/安全导航 #909
  • [x] 样式/分号 #908
  • [x] 样式/单行方法 #909
  • [x] 样式/特殊全局变量 #909
  • [x] 样式/StderrPuts #908
  • [x] 样式/字符串文字 #874
  • [x] 样式/StringLiteralsInInterpolation #909
  • [x] 样式/结构继承 #904
  • [x] 样式/符号数组 #898
  • [x] 样式/SymbolProc #898
  • [x] 样式/三元括号​​ #898
  • [x] 样式/TrailingCommaInArrayLiteral #859
  • [x] 样式/TrailingCommaInHashLiteral #859
  • [x] 样式/琐碎的附件 #898
  • [x] 样式/除非其他 #898
  • [x] 样式/字数组 #893
  • [x] 风格/尤达条件 #898
  • [x] 样式/零长度谓词 #858
  • [x] 指标/行长 #938
help wanted

最有用的评论

谢谢大家! 🚀

所有5条评论

删除此评论。 我之前的说法是错误的。 未来的开发人员在检查违规时应该注意 .rubocop_todo.yml 文件。

当前的主引用有一些 rubocop 违规行为没有被 CI 捕获。 这些问题已在https://github.com/lostisland/faraday/pull/870https://github.com/lostisland/faraday/pull/871中修复

此外,如果 PR 的名称类似于[RuboCop] fix Layout/TrailingBlankLines ,那么将它们添加到上述清单中会容易得多。

我检查了Style/GlobalVars因为唯一的冒犯者是script/generate_certs 。 一旦旧的集成测试被完全移植,集成套件的旧脚本就可以使用了,这种攻击将被神奇地修复。 在别处添加的任何其他全局变量应继续被标记为 rubocop 犯罪。

在修复Style/Globalvars之前,我还没有看过这篇文章:man_shrugging:

谢谢大家! 🚀

此页面是否有帮助?
0 / 5 - 0 等级

相关问题

luizkowalski picture luizkowalski  ·  3评论

ryanbyon picture ryanbyon  ·  3评论

subvertallchris picture subvertallchris  ·  5评论

Lewiscowles1986 picture Lewiscowles1986  ·  4评论

jeffb-stell picture jeffb-stell  ·  5评论