Refactor user theme CSS to be provided by a stylesheet endpoint #1035
reviewdog [rubocop] report
reported by reviewdog 🐶
Findings (1)
spec/controllers/user_controller_spec.rb|85 col 1| Use 2 spaces for indentation in a heredoc.
Filtered Findings (641)
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/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|53 col 17| Align .and
with .where
on line 52.
app/controllers/answer_controller.rb|57 col 17| Align .update_all
with .where
on line 52.
app/controllers/answer_controller.rb|58 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|48 col 5| Memoized variable @active_announcements
does not match method name find_active_announcements
. Use @find_active_announcements
instead.
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/profile_picture_controller.rb|13 col 7| Use flash.now
before render
.
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|41 col 7| Put a comma after the last item of a multiline hash.
app/controllers/user_controller.rb|58 col 7| Put a comma after the last item of a multiline hash.
app/controllers/user_controller.rb|88 col 9| Put a comma after the last parameter of a multiline method call.
app/controllers/user_controller.rb|100 col 7| Align &.select
with .where
on line 99.
app/controllers/user_controller.rb|101 col 7| Align &.group_by
with .where
on line 99.
app/helpers/layouts_helper.rb|5 col 5| 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|33 col 12| Align )
with (
.
app/models/answer.rb|33 col 12| Closing method call brace must be on the same line as the last argument when opening brace is on the same line as the first argument.
app/models/answer.rb|42 col 89| Use ||
instead of or
.
app/models/answer.rb|49 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.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.
db/migrate/20210811133004_add_direct_to_questions.rb|14 col 5| execute is not reversible.
db/migrate/20210811133004_add_direct_to_questions.rb|21 col 5| execute is not reversible.
db/migrate/20211219153054_create_profiles.rb|17 col 7| execute is not reversible.
db/migrate/20211219153054_create_profiles.rb|19 col 7| remove_column(without type) is not reversible.
db/migrate/20211219153054_create_profiles.rb|20 col 7| remove_column(without type) is not reversible.
db/migrate/20211219153054_create_profiles.rb|21 col 7| remove_column(without type) is not reversible.
db/migrate/20211219153054_create_profiles.rb|22 col 7| remove_column(without type) is not reversible.
db/migrate/20211219153054_create_profiles.rb|23 col 7| remove_column(without type) is not reversible.
db/migrate/20211222165159_create_mute_rules.rb|12 col 5| change_column is not reversible.
db/migrate/20220909161545_rpush_2_6_0_updates.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20220909161547_rpush_3_0_0_updates.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20220909161556_rpush_4_1_1_updates.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20220909161557_rpush_4_2_0_updates.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20221106130744_add_inbox_locked_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20221113110942_add_privacy_require_user_to_users.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20221115194933_add_privacy_hide_social_graph.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20221116201723_add_privacy_noindex_to_user.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20230108114333_add_allow_long_questions_to_profiles.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20230205145011_add_sharing_fields_to_user.rb|5 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20230205145011_add_sharing_fields_to_user.rb|6 col 5| Boolean columns should always have a default value and a NOT NULL
constraint.
db/migrate/20231028091613_move_appendable_notifications.rb|5 col 17| Do not use ::
for method calls.
db/migrate/20231028091613_move_appendable_notifications.rb|9 col 17| Do not use ::
for method calls.
lib/tasks/locale.rake|13 col 5| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/answers.rb|8 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/answers.rb|9 col 9| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/comments.rb|8 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/comments.rb|9 col 9| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/mute_rules.rb|8 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/mute_rules.rb|9 col 9| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/questions.rb|12 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/questions.rb|13 col 9| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/relationships.rb|15 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/relationships.rb|16 col 9| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/theme.rb|11 col 13| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/theme.rb|12 col 11| Put a comma after the last item of a multiline hash.
lib/use_case/data_export/user.rb|29 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/data_export/user.rb|31 col 9| Put a comma after the last item of a multiline hash.
lib/use_case/mute_rule/create.rb|12 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/mute_rule/create.rb|17 col 11| Put a comma after the last item of a multiline hash.
lib/use_case/mute_rule/destroy.rb|13 col 11| Put a comma after the last item of a multiline hash.
lib/use_case/question/create.rb|76 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/question/create.rb|102 col 11| Put a comma after the last parameter of a multiline method call.
lib/use_case/relationship/create.rb|17 col 13| Put a comma after the last item of a multiline hash.
lib/use_case/relationship/create.rb|17 col 26| Omit the hash value.
lib/use_case/relationship/create.rb|18 col 11| Put a comma after the last item of a multiline hash.
lib/use_case/relationship/destroy.rb|17 col 13| Put a comma after the last item of a multiline hash.
lib/use_case/relationship/destroy.rb|17 col 26| Omit the hash value.
lib/use_case/relationship/destroy.rb|18 col 11| Put a comma after the last item of a multiline hash.
lib/use_case/user/ban.rb|21 col 13| Put a comma after the last parameter of a multiline method call.
lib/use_case/user/ban.rb|27 col 13| Put a comma after the last parameter of a multiline method call.
lib/use_case/user/ban.rb|35 col 13| Put a comma after the last item of a multiline hash.
lib/use_case/user/ban.rb|36 col 11| Put a comma after the last item of a multiline hash.
lib/use_case/user/unban.rb|15 col 13| Put a comma after the last item of a multiline hash.
lib/use_case/user/unban.rb|15 col 26| Omit the hash value.
... (Too many findings. Dropped some findings)
Annotations
Check notice on line 115 in spec/controllers/user_controller_spec.rb
github-actions / rubocop
[rubocop] spec/controllers/user_controller_spec.rb#L85-L115 <Layout/HeredocIndentation>
Use 2 spaces for indentation in a heredoc.
Raw output
spec/controllers/user_controller_spec.rb:85:1: C: Layout/HeredocIndentation: Use 2 spaces for indentation in a heredoc.