aboutsummaryrefslogtreecommitdiffstats
path: root/.rubocop.yml
diff options
context:
space:
mode:
authorKoichi ITO <koic.ito@gmail.com>2018-01-18 12:09:16 +0900
committerKoichi ITO <koic.ito@gmail.com>2018-01-18 17:19:13 +0900
commit5ac6ec54a673e493cddf6bf2eff5b98e52b3c268 (patch)
tree1a11a6b248c977ccb1a6c0db46cc69625a04ea51 /.rubocop.yml
parentffd9902b4464719bd92d49db709d1cf8865337e3 (diff)
downloadrails-5ac6ec54a673e493cddf6bf2eff5b98e52b3c268.tar.gz
rails-5ac6ec54a673e493cddf6bf2eff5b98e52b3c268.tar.bz2
rails-5ac6ec54a673e493cddf6bf2eff5b98e52b3c268.zip
Enable autocorrect for `Lint/EndAlignment` cop
### Summary This PR changes .rubocop.yml. Regarding the code using `if ... else ... end`, I think the coding style that Rails expects is as follows. ```ruby var = if cond a else b end ``` However, the current .rubocop.yml setting does not offense for the following code. ```ruby var = if cond a else b end ``` I think that the above code expects offense to be warned. Moreover, the layout by autocorrect is unnatural. ```ruby var = if cond a else b end ``` This PR adds a setting to .rubocop.yml to make an offense warning and autocorrect as expected by the coding style. And this change also fixes `case ... when ... end` together. Also this PR itself is an example that arranges the layout using `rubocop -a`. ### Other Information Autocorrect of `Lint/EndAlignment` cop is `false` by default. https://github.com/bbatsov/rubocop/blob/v0.51.0/config/default.yml#L1443 This PR changes this value to `true`. Also this PR has changed it together as it is necessary to enable `Layout/ElseAlignment` cop to make this behavior.
Diffstat (limited to '.rubocop.yml')
-rw-r--r--.rubocop.yml4
1 files changed, 4 insertions, 0 deletions
diff --git a/.rubocop.yml b/.rubocop.yml
index a04de4b497..3b4dd79e81 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -26,6 +26,9 @@ Layout/CaseIndentation:
Layout/CommentIndentation:
Enabled: true
+Layout/ElseAlignment:
+ Enabled: true
+
Layout/EmptyLineAfterMagicComment:
Enabled: true
@@ -140,6 +143,7 @@ Style/UnneededPercentQ:
Lint/EndAlignment:
Enabled: true
EnforcedStyleAlignWith: variable
+ AutoCorrect: true
# Use my_method(my_arg) not my_method( my_arg ) or my_method my_arg.
Lint/RequireParentheses: