Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor user theme CSS to be provided by a stylesheet endpoint #1035

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Don't include theme CSS if there is no theme user present

92b0382
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

Refactor user theme CSS to be provided by a stylesheet endpoint #1035

Don't include theme CSS if there is no theme user present
92b0382
Select commit
Loading
Failed to load commit list.
GitHub Actions / rubocop failed Dec 20, 2023 in 1s

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

See this annotation in the file changed.

@github-actions 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.