Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update RuboCop empty lines layout config #8014

Merged
merged 3 commits into from
Nov 21, 2023
Merged

Update RuboCop empty lines layout config #8014

merged 3 commits into from
Nov 21, 2023

Conversation

gbp
Copy link
Member

@gbp gbp commented Nov 16, 2023

Enable all empty lines layout cops and auto correct all Ruby files.

Before this change we had at least two different styles around empty
lines so this ensure consistency across Ruby files.

  • Layout/EmptyLineAfterGuardClause
  • Layout/EmptyLineAfterMagicComment
  • Layout/EmptyLineBetweenDefs
  • Layout/EmptyLines
  • Layout/EmptyLinesAroundAccessModifier
  • Layout/EmptyLinesAroundArguments
  • Layout/EmptyLinesAroundAttributeAccessor
  • Layout/EmptyLinesAroundBeginBody
  • Layout/EmptyLinesAroundBlockBody
  • Layout/EmptyLinesAroundClassBody
  • Layout/EmptyLinesAroundMethodBody
  • Layout/EmptyLinesAroundModuleBody
  • Layout/LeadingEmptyLines
  • Layout/TrailingEmptyLines

Ignores:

  • Layout/EmptyLineAfterMultilineCondition
  • Layout/EmptyLinesAroundExceptionHandlingKeywords

Instead we use are own EmptyLineAroundRescuedExceptions cop.


[skip changelog]

@gbp gbp force-pushed the rubocop-empty-lines branch from cb520e3 to f3599c0 Compare November 17, 2023 13:54
gbp added 3 commits November 21, 2023 10:21
Enable all empty lines layout cops and auto correct all Ruby files.

Before this change we had at least two different styles around empty
lines so this ensure consistency across Ruby files.

- `Layout/EmptyLineAfterGuardClause`
- `Layout/EmptyLineAfterMagicComment`
- `Layout/EmptyLineBetweenDefs`
- `Layout/EmptyLines`
- `Layout/EmptyLinesAroundAccessModifier`
- `Layout/EmptyLinesAroundArguments`
- `Layout/EmptyLinesAroundAttributeAccessor`
- `Layout/EmptyLinesAroundBeginBody`
- `Layout/EmptyLinesAroundBlockBody`
- `Layout/EmptyLinesAroundClassBody`
- `Layout/EmptyLinesAroundMethodBody`
- `Layout/EmptyLinesAroundModuleBody`
- `Layout/LeadingEmptyLines`
- `Layout/TrailingEmptyLines`

Ignores:
- `Layout/EmptyLineAfterMultilineCondition`
- `Layout/EmptyLinesAroundExceptionHandlingKeywords`

Instead we use are own `EmptyLineAroundRescuedExceptions` cop.
Cop has been renamed
Ignore installed themes files.
@gbp gbp force-pushed the rubocop-empty-lines branch from 6554f9e to 7021aac Compare November 21, 2023 10:21
@gbp gbp merged commit e9b5d2a into develop Nov 21, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant