-
Notifications
You must be signed in to change notification settings - Fork 13.4k
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
Linting and RuboCop: Paragraph 'Convention time again' needs a little love #28821
Comments
For the first one, it probably is just not-so-fortunate omission:
"functions are verbs" -> So perhaps something like:
should work a lot better. Now thinking about this, I wonder whether "functions" should be changed to "methods" but I think my idea was to bring it as close to what-has-been with JS as possible.
You're right. This might be me coming from a different place. I'm pretty sure that at that point I read enough nasty nested There's a spectre of polymorphism hiding behind this statement as well but before Chess, the benefits of polymorphism will probably be unknown to most. They were to me for sure!
Yeah this most likely should say iterative methods or something like that. It is indeed a reference to things like enumerables. Relevant Ruby Style guide bit. |
I like these:
Tangential: |
Happy to submit a PR for this! |
Checks
Describe your suggestion
There are some issues with the first paragraph under 'Convention time again':
do_something(with_thing, another_thing)
if
statements"+s
prefer iterative approaches"each
,map
...)? Because loops are inherently an iterative approach as far as I understand (e.g. compared to recursion).Path
Ruby / Rails
Lesson Url
https://www.theodinproject.com/lessons/ruby-linting-and-rubocop
(Optional) Discord Name
ruben-vl
(Optional) Additional Comments
Seems like an easy first contribution. Will volunteer after the weekend if no-one else is open to be assigned.
The text was updated successfully, but these errors were encountered: