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

Warnings under newer rubies #4

Open
mwpastore opened this issue Oct 30, 2017 · 1 comment
Open

Warnings under newer rubies #4

mwpastore opened this issue Oct 30, 2017 · 1 comment

Comments

@mwpastore
Copy link

Getting some warnings running Hobby in test:

/Users/mwp/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/hobby-0.1.1/lib/hobby.rb:20: warning: parentheses after method name is interpreted as an argument list, not a decomposed argument
/Users/mwp/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/hobby-0.1.1/lib/hobby.rb:51: warning: `&' interpreted as argument prefix
/Users/mwp/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/hobby-0.1.1/lib/hobby/router.rb:45: warning: `*' interpreted as argument prefix
@ch1c0t
Copy link
Owner

ch1c0t commented Nov 24, 2017

I am not a fan of parentheses when they can be omitted. This is just a personal preference.

The semantics(which the warnings notify about) is intended. It is just a question of syntactic preferences, I believe.

I am not inclined to fix them. But thank you for your attentiveness.

To reproduce, I run the specs with bundle exec rspec --warnings. Otherwise, no warnings are shown.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants