Skip to content

add some search functionality #1052

add some search functionality

add some search functionality #1052

GitHub Actions / rubocop failed Oct 22, 2023 in 0s

reviewdog [rubocop] report

reported by reviewdog 🐶

Findings (14)

app/controllers/search_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/search_controller.rb|24 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/search_controller.rb|27 col 17| Prefer index_by over map { ... }.to_h.
app/controllers/search_controller.rb|27 col 51| Pass a block to to_h instead of calling map.to_h.
app/controllers/search_controller.rb|29 col 5| Avoid multi-line chains of blocks.
app/controllers/search_controller.rb|33 col 7| Align .map with .client on line 17.
app/helpers/search_helper.rb|1 col 1| Missing frozen string literal comment.
spec/helpers/search_helper_spec.rb|1 col 1| Missing frozen string literal comment.
spec/helpers/search_helper_spec.rb|1 col 9| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
spec/requests/search_spec.rb|1 col 1| Missing frozen string literal comment.
spec/requests/search_spec.rb|1 col 9| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
spec/requests/search_spec.rb|10 col 1| Extra empty line detected at block body end.
spec/views/search/index.html.erb_spec.rb|1 col 1| Missing frozen string literal comment.
spec/views/search/index.html.erb_spec.rb|1 col 9| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

Filtered Findings (767)

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/relationship_controller.rb|12 col 7| Put a comma after the last parameter of a multiline method call.
app/controllers/ajax/relationship_controller.rb|26 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/smile_controller.rb|1 col 1| Missing frozen string literal comment.
app/controllers/ajax/subscription_controller.rb|1 col 1| Missing frozen string literal comment.
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|31 col 89| Use || instead of or.
app/models/answer.rb|38 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|19 col 50| Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
app/models/question.rb|33 col 21| Use self.answers.count.positive? instead of self.answers.count > 0.
app/models/question.rb|34 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.

Annotations

Check notice on line 1 in app/controllers/search_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/search_controller.rb#L1 <Style/FrozenStringLiteralComment>

Missing frozen string literal comment.
Raw output
app/controllers/search_controller.rb:1:1: C: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

Check notice on line 24 in app/controllers/search_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/search_controller.rb#L24 <Style/TrailingCommaInArguments>

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

Check notice on line 27 in app/controllers/search_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/search_controller.rb#L27 <Rails/IndexBy>

Prefer `index_by` over `map { ... }.to_h`.
Raw output
app/controllers/search_controller.rb:27:17: C: Rails/IndexBy: Prefer `index_by` over `map { ... }.to_h`.

Check notice on line 27 in app/controllers/search_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/search_controller.rb#L27 <Style/MapToHash>

Pass a block to `to_h` instead of calling `map.to_h`.
Raw output
app/controllers/search_controller.rb:27:51: C: Style/MapToHash: Pass a block to `to_h` instead of calling `map.to_h`.

Check notice on line 32 in app/controllers/search_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/search_controller.rb#L29-L32 <Style/MultilineBlockChain>

Avoid multi-line chains of blocks.
Raw output
app/controllers/search_controller.rb:29:5: C: Style/MultilineBlockChain: Avoid multi-line chains of blocks.

Check notice on line 33 in app/controllers/search_controller.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/controllers/search_controller.rb#L33 <Layout/MultilineMethodCallIndentation>

Align `.map` with `.client` on line 17.
Raw output
app/controllers/search_controller.rb:33:7: C: Layout/MultilineMethodCallIndentation: Align `.map` with `.client` on line 17.

Check notice on line 1 in app/helpers/search_helper.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] app/helpers/search_helper.rb#L1 <Style/FrozenStringLiteralComment>

Missing frozen string literal comment.
Raw output
app/helpers/search_helper.rb:1:1: C: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

Check notice on line 1 in spec/helpers/search_helper_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/helpers/search_helper_spec.rb#L1 <Style/FrozenStringLiteralComment>

Missing frozen string literal comment.
Raw output
spec/helpers/search_helper_spec.rb:1:1: C: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

Check notice on line 1 in spec/helpers/search_helper_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/helpers/search_helper_spec.rb#L1 <Style/StringLiterals>

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Raw output
spec/helpers/search_helper_spec.rb:1:9: C: Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

Check notice on line 1 in spec/requests/search_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/requests/search_spec.rb#L1 <Style/FrozenStringLiteralComment>

Missing frozen string literal comment.
Raw output
spec/requests/search_spec.rb:1:1: C: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

Check notice on line 1 in spec/requests/search_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/requests/search_spec.rb#L1 <Style/StringLiterals>

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Raw output
spec/requests/search_spec.rb:1:9: C: Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

Check notice on line 11 in spec/requests/search_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/requests/search_spec.rb#L10-L11 <Layout/EmptyLinesAroundBlockBody>

Extra empty line detected at block body end.
Raw output
spec/requests/search_spec.rb:10:1: C: Layout/EmptyLinesAroundBlockBody: Extra empty line detected at block body end.

Check notice on line 1 in spec/views/search/index.html.erb_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/views/search/index.html.erb_spec.rb#L1 <Style/FrozenStringLiteralComment>

Missing frozen string literal comment.
Raw output
spec/views/search/index.html.erb_spec.rb:1:1: C: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

Check notice on line 1 in spec/views/search/index.html.erb_spec.rb

See this annotation in the file changed.

@github-actions github-actions / rubocop

[rubocop] spec/views/search/index.html.erb_spec.rb#L1 <Style/StringLiterals>

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Raw output
spec/views/search/index.html.erb_spec.rb:1:9: C: Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.