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

Redundant block call parentheses #2689

Merged
merged 2 commits into from
Jan 21, 2016

Conversation

rrosenblum
Copy link
Contributor

I came across an edge case that caused auto-correct to break the code. Take a look at the second test case for the example code.

@alexdowad
Copy link
Contributor

Thanks for fixing my code.

@rrosenblum rrosenblum force-pushed the redundant_block_call_parentheses branch from 6d0393f to 04ae9c7 Compare January 20, 2016 20:32
@rrosenblum
Copy link
Contributor Author

Welcome, I don't expect this to be a problem for too many people. Regardless of impact, it was worth fixing.

I came across one or two more bugs with the new release. I will try to get PRs in for them soon.

@rrosenblum rrosenblum force-pushed the redundant_block_call_parentheses branch from 04ae9c7 to 0a30b93 Compare January 20, 2016 20:44
@bbatsov
Copy link
Collaborator

bbatsov commented Jan 21, 2016

The build is failing.

@rrosenblum rrosenblum force-pushed the redundant_block_call_parentheses branch from 0a30b93 to 01c78b6 Compare January 21, 2016 17:13
bbatsov added a commit that referenced this pull request Jan 21, 2016
@bbatsov bbatsov merged commit b357b6c into rubocop:master Jan 21, 2016
@rrosenblum rrosenblum deleted the redundant_block_call_parentheses branch January 22, 2016 22:48
jonas054 added a commit to jonas054/rubocop that referenced this pull request Jan 30, 2016
Operands in expressions that are the right-hand-side in an
assignment where there's a line break after = should be
aligned with each other, regardless of configured style.
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

Successfully merging this pull request may close these issues.

3 participants