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.

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

Jump to bottom

Hash.new autocorrect leaves invalid ruby #1574

Closed
JeanMertz opened this issue Jan 17, 2015 · 0 comments
Closed

Hash.new autocorrect leaves invalid ruby #1574

JeanMertz opened this issue Jan 17, 2015 · 0 comments
Assignees
@jonas054
Labels
bug

Comments

@JeanMertz
Copy link

this:

yadayada.map { ... }.reduce Hash.new, :merge

gets auto corrected to:

yadayada.map { ... }.reduce {}, :merge

Which is invalid Ruby. Braces would fix the problem, but they didn't get added automatically.
@jonas054 jonas054 self-assigned this Jan 18, 2015
@jonas054 jonas054 added the bug label Jan 18, 2015
bbatsov added a commit that referenced this issue Jan 22, 2015
@bbatsov
Merge pull request #1590 from jonas054/1574_fix_hash_new_autocorrect
e1a533b 
[Fix #1574] Don't auto-correct Hash.new into block braces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonas054 jonas054

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jonas054 @JeanMertz