mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
5ac6ec54a6
### 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.
|
||
|---|---|---|
| .. | ||
| authorization.rb | ||
| base.rb | ||
| client_socket.rb | ||
| identification.rb | ||
| internal_channel.rb | ||
| message_buffer.rb | ||
| stream.rb | ||
| stream_event_loop.rb | ||
| subscriptions.rb | ||
| tagged_logger_proxy.rb | ||
| web_socket.rb | ||