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

Fix edge cases typing Enter before /**/ comments. #11073

Merged
merged 1 commit into from
May 11, 2016
Merged

Conversation

pawchen
Copy link
Contributor

@pawchen pawchen commented May 4, 2016

Fixes #11056
Fixes #11057

@pawchen
Copy link
Contributor Author

pawchen commented May 4, 2016

@davkean Jenkins git failed to checkout?

@davkean
Copy link
Member

davkean commented May 4, 2016

@mattwar @dotnet/roslyn-infrastructure

@jasonmalinowski
Copy link
Member

We've had this happen once or twice more recently. I thought I had broken Jenkins for my own PR...clearly not.

@jasonmalinowski
Copy link
Member

@dotnet/roslyn-ide for the review of the code itself.

@davkean davkean added the Community The pull request was submitted by a contributor who is not a Microsoft employee. label May 5, 2016
@davkean
Copy link
Member

davkean commented May 5, 2016

retest this please

@DustinCampbell
Copy link
Member

LGTM 👍

@balajikris
Copy link
Member

👍

@pawchen
Copy link
Contributor Author

pawchen commented May 11, 2016

Any extra tests need to run?

@davkean
Copy link
Member

davkean commented May 11, 2016

@DiryBoy Sorry - looks like we missed this one. Thanks for getting on these so quickly!

@davkean davkean merged commit 555f4fa into dotnet:master May 11, 2016
@pawchen
Copy link
Contributor Author

pawchen commented May 11, 2016

@davkean You always respond in seconds. It's like late at night there?

@pawchen pawchen deleted the edge branch May 11, 2016 06:23
@davkean
Copy link
Member

davkean commented May 11, 2016

@DiryBoy Not too late, just before midnight. Github being external, means that I'm always on if I'm awake. :)

@pawchen
Copy link
Contributor Author

pawchen commented May 11, 2016

@davkean Awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-already-signed Community The pull request was submitted by a contributor who is not a Microsoft employee.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants