Skip to content

Merge branch 'main' into versionup-v4.2.5 #127

Merge branch 'main' into versionup-v4.2.5

Merge branch 'main' into versionup-v4.2.5 #127

Triggered via push February 1, 2024 15:47
Status Failure
Total duration 1m 57s
Artifacts

lint-ruby.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 3 warnings
lint: app/controllers/settings/request_custom_emojis_controller.rb#L50
[Correctable] Layout/IndentationWidth: Use 2 (not 0) spaces for indentation. (https://rubystyle.guide#spaces-indentation)
lint: app/controllers/settings/request_custom_emojis_controller.rb#L69
[Correctable] Style/RedundantReturn: Redundant return detected. (https://rubystyle.guide#no-explicit-return)
lint: app/models/form/request_custom_emoji_batch.rb#L30
[Correctable] Style/CombinableLoops: Combine this loop with the previous loop.
lint: 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)
lint: app/models/form/request_custom_emoji_batch.rb#L33
[Correctable] Layout/EmptyLineAfterGuardClause: Add empty line after guard clause.
lint: app/models/form/request_custom_emoji_batch.rb#L46
[Correctable] Style/CombinableLoops: Combine this loop with the previous loop.
lint: 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)
lint: 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)
lint: app/models/form/request_custom_emoji_batch.rb#L56
[Correctable] Style/CombinableLoops: Combine this loop with the previous loop.
lint: app/models/form/request_custom_emoji_batch.rb#L56
[Correctable] Style/SymbolProc: Pass &:destroy as an argument to each instead of a block.
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: r7kamura/rubocop-problem-matchers-action@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint: 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.
lint: app/models/request_custom_emoji.rb#L53
[Correctable] Lint/UselessAccessModifier: Useless private access modifier.