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

allow new lines and comments in lambda parameters #307

Merged
merged 1 commit into from
Jul 30, 2018

Conversation

dten
Copy link

@dten dten commented Jun 17, 2018

cc #284 6 and 7

image

@dten
Copy link
Author

dten commented Jun 30, 2018

@jasonwilliams can we get some of these syntax fixes merged, the current default is far from what it will be after all these merge

@ehuss
Copy link

ehuss commented Jul 2, 2018

@dten I think Jason is on vacation for the next week, so it'll probably be delayed for a little while.

@dten
Copy link
Author

dten commented Jul 2, 2018

Doh I missed that

@jasonwilliams
Copy link
Member

Yes when I’m back I will try to merge some more of these, @dten are you able to join the official Rust discord? https://discord.gg/JTcKGAy Ping me once you’re in there.

@jasonwilliams
Copy link
Member

@dten conflict here

@dten
Copy link
Author

dten commented Jul 12, 2018

that was the world's most mundane conflict 🎉

@jasonwilliams
Copy link
Member

@dten sorry, but as i was merging in the other PRs i caused another conflict :/

@ehuss ehuss merged commit 6997829 into rust-lang:master Jul 30, 2018
@dten
Copy link
Author

dten commented Jul 31, 2018

what i didn't get was that it wasn't really a conflict, i added something to the bottom of a file, so did someone else (we both added tests) i've no idea why it would complain

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