Skip to content

Issues: rubocop/rails-style-guide

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.

Sign up for GitHub

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

22 Open 70 Closed
22 Open 70 Closed
Author
Filter by author
Label
Filter by label
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Milestones
Filter by milestone
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

Cop idea: Prefer assert_raises over assert_raise
#351 opened Oct 20, 2023 by Earlopain
10
"Redundant all" is too naive
#347 opened Oct 13, 2023 by zverok
10
Suggestion: 3-state booleans don't require a default
#343 opened Aug 29, 2023 by joerixaop
3
Suggestion: don't divide .where.not into two lines
#341 opened Aug 14, 2023 by ydakuka
Suggestion: Add notes about returned value of ActiveRecord transaction
#338 opened Jun 21, 2023 by ydakuka
Suggestion: Add notes about .none()
#337 opened Jun 21, 2023 by ydakuka
1
Are blank routes preferred or routes with /
#336 opened Jun 9, 2023 by arianf
Cop idea: prefer symbol proc to if: and unless: filter lambdas
#335 opened Apr 6, 2023 by vlad-pisanov
2
Add delete_all to the list of methods that skip model validations
#332 opened Dec 19, 2022 by etherbob
2
Suggestion: Use string literals instead of named routes or URL helpers in tests
#328 opened Oct 22, 2022 by ohbarye
2
Suggestion: Avoid render ... and return
#327 opened Sep 9, 2022 by andyw8
4
Use more specific predicates instead of vague blank? and present?
#291 opened Sep 18, 2021 by pirj
2
7 Standard Controller Actions
#290 opened Sep 13, 2021 by mollerhoj
11
Controller action ordering
#289 opened Sep 11, 2021 by mollerhoj
11
strict_loading_by_default ?
#285 opened Apr 29, 2021 by andyw8
1
Clarify the reasoning for Define Model Class Migrations
#284 opened Apr 9, 2021 by pirj
8
The current order by id recommendation may be subtly introducing bugs.
#274 opened Nov 11, 2020 by kitsunde
9
Guidance for using html_safe
#260 opened Feb 11, 2020 by andyw8
4
New cop: Every AR find must either be in a conditional or be a bang
#333 opened Dec 6, 2019 by andyw8
1
Cop suggestion: PreferCreateJoinTable
#334 opened Jan 3, 2019 by connorshea
4
Faster Rails: Use exists instead of present? to check if a Record Exists
#232 opened Aug 27, 2018 by zoras
9
read_attribute(:attr) and self[:attr] not equivalent
#155 opened Sep 1, 2015 by amar47shah
12
ProTip! Type g p on any issue or pull request to go back to the pull request listing page.