Skip to content

Merge master to features/embeddedRegex#27707

Merged
3 commits merged intofeatures/embeddedRegexfrom
merges/master-to-features/embeddedRegex
Jun 11, 2018
Merged

Merge master to features/embeddedRegex#27707
3 commits merged intofeatures/embeddedRegexfrom
merges/master-to-features/embeddedRegex

Conversation

@dotnet-bot
Copy link
Copy Markdown
Collaborator

This is an automatically generated pull request from master into features/embeddedRegex.

git fetch --all
git checkout merges/master-to-features/embeddedRegex
git reset --hard upstream/features/embeddedRegex
git merge upstream/master
# Fix merge conflicts
git commit
git push merges/master-to-features/embeddedRegex --force

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request.

David Karlaš and others added 3 commits June 8, 2018 15:23
…Windows line ending

I used `\n` and `\r\n` in string instead `@""` because as far as I understand on Windows/CI code is checked out with `git config core.autocrlf` which means it modifies line endings...
…t infrastructure assumption that all code is `\r\n`
Fix #26358: AbstractDocumentationCommentCommandHandler has hardcoded Windows line ending
Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approval

@ghost ghost merged commit 494a8ef into features/embeddedRegex Jun 11, 2018
@ghost ghost deleted the merges/master-to-features/embeddedRegex branch June 11, 2018 17:30
@dotnet-bot dotnet-bot restored the merges/master-to-features/embeddedRegex branch June 11, 2018 18:01
@dotnet-bot dotnet-bot deleted the merges/master-to-features/embeddedRegex branch June 11, 2018 18:01
@dotnet-bot dotnet-bot restored the merges/master-to-features/embeddedRegex branch June 11, 2018 19:00
@dotnet-bot dotnet-bot deleted the merges/master-to-features/embeddedRegex branch June 11, 2018 19:00
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants