Skip to content

/ ruby-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.

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

64 Open 232 Closed
64 Open 232 Closed
Author
Filter by author
Label
Filter by label
Use alt + click/return to exclude labels.
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
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀
Empty line around nesting blocks of the same indentation
#823 opened Jul 2, 2020 by joallard
`alias_method` should be preferred to `alias`
#821 opened Jun 13, 2020 by marcandre
2
Recommend using allowlist/denylist instead of whitelist/blacklist
#820 opened Jun 8, 2020 by bai
3
Guard clause in a middle of function?
#813 opened Apr 3, 2020 by sandric
Recommend calling super in initialize Approved
#809 opened Mar 27, 2020 by headius
3
Avoiding use of class references built via strings/`constantize`
#806 opened Feb 28, 2020 by Skipants
5
Prefer `drop` over `slice` (or `shift`) to retrieve all but the first array element New Rule
#794 opened Feb 27, 2020 by andyw8
Revise "No Braces Options Hash" rule
#788 opened Jan 13, 2020 by pocke
Use `#fetch` for arrays? New Rule
#781 opened Nov 17, 2019 by andyw8
Introduce a reference .editorconfig file
#774 opened Aug 27, 2019 by marcoXbresciani
2
Sugesstion: method name should make clear how many results are expected New Rule
#753 opened Jun 12, 2019 by dennisvandehoef
1
Restore references to original discussions and sources Help Wanted
#749 opened Jun 3, 2019 by pirj
1
Suggestion: Prefer to _not_ use two argument version of raise Approved Existing Rule
#747 opened May 23, 2019 by rhys-vdw
3
`X = Struct.new(:x) do` is bad Approved
#744 opened Apr 25, 2019 by iGEL
1
style guide doesn't follow Style/RescueStandardError New Rule
#719 opened May 29, 2018 by Fryguy
2
Add information about Time.parse DateTime.parse to Date & Time section? New Rule
#716 opened May 15, 2018 by aried3r
1
Recommendation for a comment that contains a very long url Approved Existing Rule
#712 opened Apr 23, 2018 by jonahx
5
Numeric#nonzero? considered harmful? Approved New Rule
#711 opened Apr 12, 2018 by BigDanG
9
Empty line between methods vs no nested conditionals
#710 opened Apr 10, 2018 by gutekk
2
Using the safe navigation operator with other operators?
#705 opened Feb 16, 2018 by arempe93
2
assignment in conditional ternary New Rule
#704 opened Feb 16, 2018 by MathieuDerelle
3
positive? vs > 0 New Rule
#701 opened Feb 10, 2018 by Ana06
3
`&&=` is not reliable (maybe only for some versions)
#691 opened Dec 13, 2017 by EdmundLeex
2
Favor standard lib exceptions over new classes needs clarification Approved Existing Rule
#690 opened Dec 9, 2017 by cwjenkins
1
Ternary branch text seems too strong Approved Existing Rule
#686 opened Dec 3, 2017 by pochmann
1
ProTip! Mix and match filters to narrow down what you’re looking for.
You can’t perform that action at this time.