Skip to content

Move reactions functionality to Turbo Streams #32

Move reactions functionality to Turbo Streams

Move reactions functionality to Turbo Streams #32

GitHub Actions / rubocop failed Nov 5, 2023 in 1s

reviewdog [rubocop] report

reported by reviewdog 🐶

Findings (4)

app/controllers/reactions_controller.rb|19 col 27| Use find(params[:id]) instead of find_by!(id: params[:id]).
app/controllers/reactions_controller.rb|23 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/reactions_controller.rb|43 col 27| Use find(params[:id]) instead of find_by!(id: params[:id]).
app/controllers/reactions_controller.rb|47 col 7| Put a comma after the last parameter of a multiline method call.

Filtered Findings (693)

Brewfile|1 col 1| Missing frozen string literal comment.
app/controllers/admin/announcement_controller.rb|26 col 3| Action edit should appear before create.
app/controllers/admin/dashboard_controller.rb|11 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax/comment_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/ajax/comment_controller.rb|32 col 43| Use || instead of or.
app/controllers/ajax/comment_controller.rb|32 col 84| Use || instead of or.
app/controllers/ajax/inbox_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/ajax/inbox_controller.rb|31 col 39| Use find_each instead of each.
app/controllers/ajax/inbox_controller.rb|31 col 44| Pass &:remove as an argument to each instead of a block.
app/controllers/ajax/inbox_controller.rb|46 col 33| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/inbox_controller.rb|48 col 36| Align .where with .inboxes on line 47.
app/controllers/ajax/inbox_controller.rb|49 col 19| Pass &:remove as an argument to each instead of a block.
app/controllers/ajax/list_controller.rb|18 col 87| Omit the hash value.
app/controllers/ajax/moderation_controller.rb|34 col 19| Use find_by! instead of dynamic find_by_screen_name!.
app/controllers/ajax/moderation_controller.rb|35 col 30| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/moderation_controller.rb|65 col 7| Align the keys and values of a hash literal if they span more than one line.
app/controllers/ajax/moderation_controller.rb|65 col 15| Omit the hash value.
app/controllers/ajax/moderation_controller.rb|66 col 7| Align the keys and values of a hash literal if they span more than one line.
app/controllers/ajax/moderation_controller.rb|66 col 15| Omit the hash value.
app/controllers/ajax/moderation_controller.rb|67 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/ajax/moderation_controller.rb|67 col 38| Closing method call brace must be on the line after the last argument when opening brace is on a separate line from the first argument.
app/controllers/ajax/moderation_controller.rb|82 col 33| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/moderation_controller.rb|84 col 19| Use find_by! instead of dynamic find_by_screen_name!.
app/controllers/ajax/moderation_controller.rb|87 col 19| %w-literals should be delimited by [ and ].
app/controllers/ajax/moderation_controller.rb|97 col 19| Space inside { missing.
app/controllers/ajax/moderation_controller.rb|97 col 20| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/moderation_controller.rb|97 col 31| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/moderation_controller.rb|97 col 46| Space inside } missing.
app/controllers/ajax/question_controller.rb|9 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/ajax/question_controller.rb|9 col 21| Omit the hash value.
app/controllers/ajax/question_controller.rb|17 col 3| Action create should appear before destroy.
app/controllers/ajax/question_controller.rb|27 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax/question_controller.rb|34 col 9| Put a comma after the last parameter of a multiline method call.
app/controllers/ajax/question_controller.rb|44 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/ajax/report_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/ajax/report_controller.rb|14 col 12| %w-literals should be delimited by [ and ].
app/controllers/ajax/report_controller.rb|22 col 7| Indent when as deep as case.
app/controllers/ajax/report_controller.rb|22 col 12| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/report_controller.rb|23 col 9| Use find_by! instead of dynamic find_by_screen_name!.
app/controllers/ajax/report_controller.rb|24 col 7| Indent when as deep as case.
app/controllers/ajax/report_controller.rb|24 col 12| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/report_controller.rb|26 col 7| Indent when as deep as case.
app/controllers/ajax/report_controller.rb|26 col 12| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/report_controller.rb|28 col 7| Indent when as deep as case.
app/controllers/ajax/report_controller.rb|28 col 12| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/ajax/report_controller.rb|30 col 7| Duplicate branch body detected.
app/controllers/ajax/report_controller.rb|32 col 7| end at 32, 6 is not aligned with case at 21, 13.
app/controllers/ajax/web_push_controller.rb|36 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/ajax_controller.rb|17 col 9| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|30 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|42 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|54 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|66 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|78 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|90 col 7| Put a comma after the last item of a multiline hash.
app/controllers/ajax_controller.rb|102 col 7| Put a comma after the last item of a multiline hash.
app/controllers/anonymous_block_controller.rb|17 col 7| Align the keys and values of a hash literal if they span more than one line.
app/controllers/anonymous_block_controller.rb|18 col 7| Align the keys and values of a hash literal if they span more than one line.
app/controllers/anonymous_block_controller.rb|20 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/answer_controller.rb|55 col 17| Align .and with .where on line 54.
app/controllers/answer_controller.rb|59 col 17| Align .update_all with .where on line 54.
app/controllers/answer_controller.rb|60 col 18| Avoid using touch because it skips validations.
app/controllers/application_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/application_controller.rb|28 col 5| Use a guard clause (return unless current_user.present? && current_user.banned?) instead of wrapping the code inside a conditional expression.
app/controllers/application_controller.rb|33 col 7| Favor modifier unless usage when having a single-line body. Another good alternative is the usage of control flow &&/||.
app/controllers/application_controller.rb|37 col 11| Annotation keywords like TODO should be all upper case, followed by a colon, and a space, then a note describing the problem.
app/controllers/application_controller.rb|46 col 5| Memoized variable @active_announcements does not match method name find_active_announcements. Use @find_active_announcements instead.
app/controllers/discover_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/discover_controller.rb|5 col 5| Favor modifier unless usage when having a single-line body. Another good alternative is the usage of control flow &&/||.
app/controllers/discover_controller.rb|11 col 60| Do not use Time.now without zone. Use one of Time.zone.now, Time.current, Time.now.in_time_zone, Time.now.utc, Time.now.getlocal, Time.now.xmlschema, Time.now.iso8601, Time.now.jisx0301, Time.now.rfc3339, Time.now.httpdate, Time.now.to_i, Time.now.to_f instead.
app/controllers/discover_controller.rb|12 col 59| Do not use Time.now without zone. Use one of Time.zone.now, Time.current, Time.now.in_time_zone, Time.now.utc, Time.now.getlocal, Time.now.xmlschema, Time.now.iso8601, Time.now.jisx0301, Time.now.rfc3339, Time.now.httpdate, Time.now.to_i, Time.now.to_f instead.
app/controllers/discover_controller.rb|13 col 64| Do not use Time.now without zone. Use one of Time.zone.now, Time.current, Time.now.in_time_zone, Time.now.utc, Time.now.getlocal, Time.now.xmlschema, Time.now.iso8601, Time.now.jisx0301, Time.now.rfc3339, Time.now.httpdate, Time.now.to_i, Time.now.to_f instead.
app/controllers/discover_controller.rb|21 col 50| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/discover_controller.rb|21 col 88| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|22 col 9| Align where with Question.select('user_id, COUNT(*) AS question_count'). on line 21.
app/controllers/discover_controller.rb|22 col 38| Do not use Time.now without zone. Use one of Time.zone.now, Time.current, Time.now.in_time_zone, Time.now.utc, Time.now.getlocal, Time.now.xmlschema, Time.now.iso8601, Time.now.jisx0301, Time.now.rfc3339, Time.now.httpdate, Time.now.to_i, Time.now.to_f instead.
app/controllers/discover_controller.rb|22 col 54| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|23 col 9| Align where with Question.select('user_id, COUNT(*) AS question_count'). on line 21.
app/controllers/discover_controller.rb|23 col 42| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|24 col 9| Align group with Question.select('user_id, COUNT(*) AS question_count'). on line 21.
app/controllers/discover_controller.rb|24 col 24| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|25 col 9| Align order with Question.select('user_id, COUNT(*) AS question_count'). on line 21.
app/controllers/discover_controller.rb|25 col 15| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/discover_controller.rb|25 col 32| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|26 col 9| Align reverse_order with Question.select('user_id, COUNT(*) AS question_count'). on line 21.
app/controllers/discover_controller.rb|30 col 46| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/discover_controller.rb|30 col 82| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|31 col 9| Align where with Answer.select('user_id, COUNT(*) AS answer_count'). on line 30.
app/controllers/discover_controller.rb|31 col 38| Do not use Time.now without zone. Use one of Time.zone.now, Time.current, Time.now.in_time_zone, Time.now.utc, Time.now.getlocal, Time.now.xmlschema, Time.now.iso8601, Time.now.jisx0301, Time.now.rfc3339, Time.now.httpdate, Time.now.to_i, Time.now.to_f instead.
app/controllers/discover_controller.rb|31 col 54| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|32 col 9| Align group with Answer.select('user_id, COUNT(*) AS answer_count'). on line 30.
app/controllers/discover_controller.rb|32 col 24| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|33 col 9| Align order with Answer.select('user_id, COUNT(*) AS answer_count'). on line 30.
app/controllers/discover_controller.rb|33 col 15| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/controllers/discover_controller.rb|33 col 30| Place the . on the next line, together with the method name.
app/controllers/discover_controller.rb|34 col 9| Align reverse_order with Answer.select('user_id, COUNT(*) AS answer_count'). on line 30.
app/controllers/inbox_controller.rb|12 col 57| Unnecessary disabling of Rails/ActionControllerFlashBeforeRender.
app/controllers/inbox_controller.rb|32 col 33| Put a comma after the last parameter of a multiline method call.
app/controllers/inbox_controller.rb|94 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/manifests_controller.rb|24 col 7| Put a comma after the last item of a multiline hash.
app/controllers/manifests_controller.rb|36 col 14| Omit the hash value.
app/controllers/manifests_controller.rb|37 col 14| Omit the hash value.
app/controllers/manifests_controller.rb|41 col 11| Put a comma after the last item of a multiline hash.
app/controllers/manifests_controller.rb|43 col 7| Put a comma after the last item of a multiline hash.
app/controllers/moderation_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/notifications_controller.rb|11 col 5| Put a comma after the last item of a multiline hash.
app/controllers/notifications_controller.rb|29 col 18| Avoid using touch because it skips validations.
app/controllers/notifications_controller.rb|60 col 32| Omit the hash value.
app/controllers/notifications_controller.rb|60 col 47| Omit the hash value.
app/controllers/settings/privacy_controller.rb|16 col 52| Put a comma after the last parameter of a multiline method call.
app/controllers/settings/privacy_controller.rb|18 col 7| Use flash.now before render.
app/controllers/settings/privacy_controller.rb|20 col 7| Use flash.now before render.
app/controllers/settings/profile_controller.rb|12 col 7| Use flash.now before render.
app/controllers/settings/profile_controller.rb|14 col 7| Use flash.now before render.
app/controllers/settings/profile_picture_controller.rb|8 col 146| Put a comma after the last parameter of a multiline method call.
app/controllers/settings/sharing_controller.rb|11 col 52| Put a comma after the last parameter of a multiline method call.
app/controllers/settings/two_factor_authentication/otp_authentication_controller.rb|13 col 97| Tagging a string as html safe may be a security risk.
app/controllers/user/registrations_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/user/registrations_controller.rb|6 col 42| Space missing to the left of {.
app/controllers/user/registrations_controller.rb|21 col 40| Space missing to the left of {.
app/controllers/user_controller.rb|28 col 7| Put a comma after the last item of a multiline hash.
app/controllers/user_controller.rb|45 col 7| Put a comma after the last item of a multiline hash.
app/controllers/user_controller.rb|75 col 9| Put a comma after the last parameter of a multiline method call.
app/controllers/user_controller.rb|87 col 7| Align &.select with .where on line 86.
app/controllers/user_controller.rb|88 col 7| Align &.group_by with .where on line 86.
app/helpers/layouts_helper.rb|5 col 5| Do not use instance variables in helpers.
app/helpers/theme_helper.rb|78 col 7| Do not prefix reader method names with get_.
app/helpers/theme_helper.rb|79 col 8| Do not use instance variables in helpers.
app/helpers/theme_helper.rb|82 col 11| Do not use instance variables in helpers.
app/helpers/theme_helper.rb|87 col 9| Do not use instance variables in helpers.
app/helpers/theme_helper.rb|89 col 11| Do not use instance variables in helpers.
app/helpers/theme_helper.rb|92 col 11| Do not use instance variables in helpers.
app/helpers/theme_helper.rb|97 col 9| Do not use instance variables in helpers.
app/mailers/retromail.rb|1 col 1| Missing frozen string literal comment.
app/mailers/retromail.rb|4 col 26| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/mailers/retromail.rb|4 col 57| Do not use space inside array brackets.
app/mailers/retromail.rb|4 col 58| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/mailers/retromail.rb|4 col 72| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/mailers/retromail.rb|4 col 83| Do not use space inside array brackets.
app/mailers/retromail.rb|5 col 10| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/mailers/retromail.rb|7 col 39| Surrounding space missing in default value assignment.
app/mailers/retromail.rb|15 col 52| Surrounding space missing in default value assignment.
app/models/announcement.rb|1 col 1| Missing frozen string literal comment.
app/models/announcement.rb|24 col 5| Use a guard clause (return unless starts_at > ends_at) instead of wrapping the code inside a conditional expression.
app/models/announcement.rb|24 col 5| Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.
app/models/answer.rb|1 col 1| Missing frozen string literal comment.
app/models/answer.rb|25 col 89| Use || instead of or.
app/models/answer.rb|32 col 50| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/comment.rb|1 col 1| Missing frozen string literal comment.
app/models/comment.rb|4 col 23| Remove explicit presence validation for user_id.
app/models/comment.rb|5 col 25| Remove explicit presence validation for answer_id.
app/models/comment.rb|16 col 50| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/concerns/cursor_paginatable.rb|28 col 77| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/concerns/cursor_paginatable.rb|29 col 7| Add a comment block showing its appearance if interpolated.
app/models/list.rb|23 col 42| Omit the hash value.
app/models/list.rb|27 col 40| Omit the hash value.
app/models/list_member.rb|1 col 1| Missing frozen string literal comment.
app/models/profile.rb|1 col 1| Missing frozen string literal comment.
app/models/profile.rb|11 col 5| Use if website.present? instead of unless website.blank?.
app/models/question.rb|1 col 1| Missing frozen string literal comment.
app/models/question.rb|13 col 50| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/question.rb|27 col 21| Use self.answers.count.positive? instead of self.answers.count > 0.
app/models/question.rb|28 col 5| Add empty line after guard clause.
app/models/relationship.rb|1 col 1| Missing frozen string literal comment.
app/models/relationship.rb|2 col 35| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/relationship.rb|3 col 35| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/relationship.rb|4 col 25| Remove explicit presence validation for source_id.
app/models/relationship.rb|5 col 25| Remove explicit presence validation for target_id.
app/models/report.rb|1 col 1| Missing frozen string literal comment.
app/models/report.rb|5 col 23| Remove explicit presence validation for user_id.
app/models/report.rb|8 col 14| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/report.rb|8 col 27| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/report.rb|30 col 56| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/reports/answer.rb|1 col 1| Missing frozen string literal comment.
app/models/reports/answer.rb|2 col 4| Final newline missing.
app/models/reports/comment.rb|1 col 1| Missing frozen string literal comment.
app/models/reports/comment.rb|2 col 4| Final newline missing.
app/models/reports/question.rb|1 col 1| Missing frozen string literal comment.
app/models/reports/question.rb|2 col 4| Final newline missing.
app/models/reports/user.rb|1 col 1| Missing frozen string literal comment.
app/models/reports/user.rb|2 col 4| Final newline missing.
app/models/role.rb|4 col 3| Prefer has_many :through to has_and_belongs_to_many.
app/models/role.rb|8 col 14| Align the keys and values of a hash literal if they span more than one line.
app/models/theme.rb|1 col 1| Missing frozen string literal comment.
app/models/theme.rb|6 col 3| Prefer the new style validations validates :column, numericality: value over validates_numericality_of.
app/models/theme.rb|7 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|8 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|9 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|10 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|11 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|12 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|13 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|14 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|14 col 35| Trailing whitespace detected.
app/models/theme.rb|15 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|15 col 31| Trailing whitespace detected.
app/models/theme.rb|16 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|17 col 5| Align the arguments of a method call if they span more than one line.
app/models/theme.rb|21 col 6| Prefer string interpolation to string concatenation.
app/models/theme.rb|21 col 6| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/theme.rb|21 col 13| Prefer string interpolation to string concatenation.
app/models/theme.rb|21 col 13| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/theme.rb|25 col 6| Prefer string interpolation to string concatenation.
app/models/theme.rb|25 col 6| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/theme.rb|25 col 13| Prefer string interpolation to string concatenation.
app/models/theme.rb|25 col 13| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/totp_recovery_code.rb|1 col 1| Missing frozen string literal comment.
app/models/totp_recovery_code.rb|9 col 46| Space inside { missing.
app/models/totp_recovery_code.rb|9 col 53| Omit the hash value.
app/models/totp_recovery_code.rb|9 col 96| Space inside } missing.
app/models/user/ban_methods.rb|15 col 7| Put a comma after the last parameter of a multiline method call.
app/models/user/ban_methods.rb|27 col 19| Omit the hash value.
app/models/user/ban_methods.rb|28 col 7| Put a comma after the last parameter of a multiline method call.
app/models/user/ban_methods.rb|28 col 19| Omit the hash value.
app/models/user_ban.rb|1 col 1| Missing frozen string literal comment.
app/services/shared_markers.rb|13 col 23| Omit the hash value.
app/services/shared_markers.rb|18 col 33| Put a comma after the last item of a multiline hash.
app/services/shared_markers.rb|23 col 45| Tagging a string as html safe may be a security risk.
app/uploaders/base_uploader.rb|1 col 1| Missing frozen string literal comment.
app/uploaders/profile_header_uploader.rb|1 col 1| Missing frozen string literal comment.
app/uploaders/profile_picture_uploader.rb|1 col 1| Missing frozen string literal comment.
app/workers/export_worker.rb|19 col 7| Put a comma after the last parameter of a multiline method call.
config.ru|1 col 1| Missing frozen string literal comment.
config.ru|3 col 9| Remove redundant ::.
config.ru|3 col 16| Use expand_path('config/environment', __dir__) instead of expand_path('../config/environment', __FILE__).
config.ru|3 col 28| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
config.ru|3 col 52| Unnecessary spacing detected.
db/migrate/20140801174930_create_questions.rb|7 col 7| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20141110210154_create_inboxes.rb|8 col 7| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20141130180152_rename_columns_in_answers.rb|6 col 5| remove_column(without type) is not reversible.
db/migrate/20141130180152_rename_columns_in_answers.rb|7 col 5| change_column is not reversible.
db/migrate/20141208111714_change_answer_content_column_type.rb|6 col 7| change_table(with change) is not reversible.
db/migrate/20141213182609_create_notifications.rb|9 col 7| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20141228202825_add_deleted_to_reports.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20141229105013_add_profile_picture_processing_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150102231343_add_supporter_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150103200732_add_privacy_options_to_users.rb|11 col 7| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150112210754_add_banned_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150112210755_create_groups.rb|9 col 7| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150125191224_add_blogger_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150419201122_add_contributor_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150421120557_add_is_active_to_subscriptions.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150508144336_add_attachment_profile_header_to_users.rb|10 col 7| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150619123121_add_translator_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL constraint.
db/migrate/20150704072402_change_default_value_of_locale.rb|5 col 5| change_column is not reversible.
db/migrate/20200425194536_remove_unused_profile_flags.rb|5 col 5| remove_column(without type) is not reversible.
db/migrate/20200425194536_remove_unused_profile_flags.rb|6 col 5| remove_column(without type) is not reversible.
db/migrate/20200425194536_remove_unused_profile_flags.rb|7 col 5| remove_column(without type) is not reversible.
db/migrate/20200425194536_remove_unused_profile_flags.rb|8 col 5| remove_column(without type) is not reversible.
db/migrate/20200425194536_remove_unused_profile_flags.rb|9 col 5| remove_column(without type) is not reversible.
db/migrate/20200425194536_remove_unused_profile_flags.rb|10 col 5| remove_column(without type) is not reversible.
db/migrate/20200517192431_remove_paperclip_fields.rb|5 col 5| remove_column(without type) is not reversible.
db/migrate/20200517192431_remove_paperclip_fields.rb|6 col 5| remove_column(without type) is not reversible.
db/migrate/20200517192431_remove_paperclip_fields.rb|7 col 5| remove_column(without type) is not reversible.
db/migrate/20200517192431_remove_paperclip_fields.rb|8 col 5| remove_column(without type) is not reversible.
db/migrate/20200517192431_remove_paperclip_fields.rb|9 col 5| remove_column(without type) is not reversible.
db/migrate/20200517192431_remove_paperclip_fields.rb|10 col 5| remove_column(without type) is not reversible.
db/migrate/20200525145144_rename_groups_to_lists.rb|15 col 5| remove_index(without column) is not reversible.
db/migrate/20200525145144_rename_groups_to_lists.rb|16 col 5| remove_index(without column) is not reversible.
db/migrate/20200525145144_rename_groups_to_lists.rb|17 col 5| remove_index(without column) is not reversible.
db/migrate/20200525145144_rename_groups_to_lists.rb|18 col 5| remove_index(without column) is not reversible.
db/migrate/20200704163504_use_timestamped_ids.rb|16 col 13| Use <<~SQL.squish instead of <<~SQL.
db/migrate/20201101155648_create_totp_recovery_codes.rb|5 col 5| Add timestamps when creating a new table.
db/migrate/20210811133004_add_direct_to_questions.rb|8 col 5| execute is not reversible.
... (Too many findings. Dropped some findings)

Annotations

Check notice on line 19 in app/controllers/reactions_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/reactions_controller.rb#L19 <Rails/FindById>

Use `find(params[:id])` instead of `find_by!(id: params[:id])`.
Raw output
app/controllers/reactions_controller.rb:19:27: C: Rails/FindById: Use `find(params[:id])` instead of `find_by!(id: params[:id])`.

Check notice on line 23 in app/controllers/reactions_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/reactions_controller.rb#L23 <Style/TrailingCommaInArguments>

Put a comma after the last parameter of a multiline method call.
Raw output
app/controllers/reactions_controller.rb:23:7: C: Style/TrailingCommaInArguments: Put a comma after the last parameter of a multiline method call.

Check notice on line 43 in app/controllers/reactions_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/reactions_controller.rb#L43 <Rails/FindById>

Use `find(params[:id])` instead of `find_by!(id: params[:id])`.
Raw output
app/controllers/reactions_controller.rb:43:27: C: Rails/FindById: Use `find(params[:id])` instead of `find_by!(id: params[:id])`.

Check notice on line 47 in app/controllers/reactions_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/reactions_controller.rb#L47 <Style/TrailingCommaInArguments>

Put a comma after the last parameter of a multiline method call.
Raw output
app/controllers/reactions_controller.rb:47:7: C: Style/TrailingCommaInArguments: Put a comma after the last parameter of a multiline method call.