New features
- #3560: Add a configuration option
empty_lines_except_namespacetoStyle/EmptyLinesAroundClassBodyandStyle/EmptyLinesAroundModuleBody. (@legendetm) - #3370: Add new
Rails/HttpPositionalArgumentscop to check your Rails 5 test code for existence of positional args usage. (@logicminds) - #3510: Add a configuration option,
ConvertCodeThatCanStartToReturnNil, toStyle/SafeNavigationto check for code that could start returningnilif safe navigation is used. (@rrosenblum) - Add a new
AllCops/StyleGuideBaseURLsetting that allows the use of relative paths and/or fragments within each cop'sStyleGuidesetting, to make forking of custom style guides easier. (@scottmatthewman) - #3566: Add new
Metric/BlockLengthcop to ensure blocks don't get too long. (@savef) - #3428: Add support for configuring
Style/PreferredHashMethodswith eithershortorverbosestyle method names. (@abrom) - #3455: Add new
Rails/DynamicFindBycop. (@pocke) - #3542: Add a configuration option,
IgnoreCopDirectives, toMetrics/LineLengthto stop cop directives (# rubocop:disable Metrics/AbcSize) from being counted when considering line length. (@jmks) - Add new
Rails/DelegateAllowBlankcop. (@connorjacobsen) - Add new
Style/MultilineMemoizationcop. (@drenmi)
Bug fixes
- #3103: Make
Style/ExtraSpacingcop register an offense for extra spaces present in single-line hash literals. (@tcdowney) - #3513: Fix false positive in
Style/TernaryParenthesesfor a ternary with ranges. (@dreyks) - #3520: Fix regression causing
Lint/AssignmentInConditionfalse positive. (@savef) - #3514: Make
Style/VariableNumbercop not register an offense when valid normal case variable names have an integer after the first_. (@b-t-g) - #3516: Make
Style/VariableNumbercop not register an offense when valid normal case variable names have an integer in the middle. (@b-t-g) - #3436: Make
Rails/SaveBangcop not register an offense when return value of a non-bang method is returned by the parent method. (@coorasse) - #3540: Fix
Style/GuardClauseto register offense for instance and singleton methods. (@tejasbubane) - #3311: Detect incompatibilities with the external encoding to prevent bad autocorrections in
Style/StringLiterals. (@deivid-rodriguez) - #3499: Ensure
Lint/UnusedBlockArgumentdoesn't make recommendations that would change arity for methods defined using#define_method. (@drenmi) - #3430: Fix exception in
Performance/RedundantMergewhen inspecting a#merge!with implicit receiver. (@drenmi) - #3411: Avoid auto-correction crash for single
wheninPerformance/CaseWhenSplat. (@jonas054) - #3286: Allow
self.a, self.b = b, ainStyle/ParallelAssignment. (@jonas054) - #3419: Report offense for
unless x.nil?inStyle/NonNilCheckifIncludeSemanticChangesistrue. (@jonas054) - #3382: Avoid auto-correction crash for multiple elsifs in
Style/EmptyElse. (@lumeet) - #3334: Do not register an offense for a literal space (
\s) inStyle/UnneededCapitalW. (@rrosenblum) - #3390: Fix SaveBang cop for multiple conditional. (@tejasbubane)
- #3577: Fix
Style/RaiseArgsnot allowing compact raise with splatted args. (@savef) - #3578: Fix safe navigation method call counting in
Metrics/AbcSize. (@savef) - #3592: Fix
Style/RedundantParenthesesfor indexing with literals. (@thegedge) - #3597: Fix the autocorrect of
Performance/CaseWhenSplatwhen trying to rearange splat expanded variables to the end of a when condition. (@rrosenblum)
Changes
- #3512: Change error message of
Lint/UnneededSplatExpansionfor array in method parameters. (@tejasbubane) - #3510: Fix some issues with
Style/SafeNavigation. Fix auto-correct of multiline if expressions, and do not register an offense for scenarios using||and ternary expression. (@rrosenblum) - #3503: Change misleading message of
Style/EmptyLinesAroundAccessModifier. (@bquorning) - #3407: Turn off autocorrect for unsafe rules by default. (@ptarjan)
- #3521: Turn off autocorrect for
Security/JSONLoadby default. (@savef) - #2903:
Style/RedundantReturnlooks for redundantreturninside conditional branches. (@lumeet)