Merge pull request #1398 from Kyutech-C3/version-up-v4.2.8 #133
Annotations
10 errors and 2 warnings
|
Run rubocop:
app/controllers/settings/request_custom_emojis_controller.rb#L50
[Correctable] Layout/IndentationWidth: Use 2 (not 0) spaces for indentation. (https://rubystyle.guide#spaces-indentation)
|
|
Run rubocop:
app/controllers/settings/request_custom_emojis_controller.rb#L69
[Correctable] Style/RedundantReturn: Redundant return detected. (https://rubystyle.guide#no-explicit-return)
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L30
[Correctable] Style/CombinableLoops: Combine this loop with the previous loop.
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L31
[Correctable] Style/IfUnlessModifier: Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||. (https://rubystyle.guide#if-as-a-modifier)
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L33
[Correctable] Layout/EmptyLineAfterGuardClause: Add empty line after guard clause.
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L46
[Correctable] Style/CombinableLoops: Combine this loop with the previous loop.
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L47
[Correctable] Style/IfUnlessModifier: Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||. (https://rubystyle.guide#if-as-a-modifier)
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L47
[Correctable] Style/NumericPredicate: Use request_custom_emoji.state.zero? instead of request_custom_emoji.state == 0. (https://rubystyle.guide#predicate-methods)
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L56
[Correctable] Style/CombinableLoops: Combine this loop with the previous loop.
|
|
Run rubocop:
app/models/form/request_custom_emoji_batch.rb#L56
[Correctable] Style/SymbolProc: Pass &:destroy as an argument to each instead of a block.
|
|
Run rubocop:
app/mailers/application_mailer.rb#L24
Lint/DuplicateMethods: Method ApplicationMailer#set_autoreply_headers! is defined at both app/mailers/application_mailer.rb:18 and app/mailers/application_mailer.rb:24.
|
|
Run rubocop:
app/models/request_custom_emoji.rb#L53
[Correctable] Lint/UselessAccessModifier: Useless private access modifier.
|
Loading