Skip to content

Conversation

@cdmihai
Copy link
Contributor

@cdmihai cdmihai commented Nov 21, 2016

CodeDom is not available on .net core, so copy over its string quoting implementation.
Ideally, all this code should be replaced with crossplatform Roselyn

Fixes #1294

CodeDom is not available on .net core, so copy over its string quoting
implementation.
Ideally, all this code should be replaced with crossplatform Roselyn

Fixes dotnet#1294
@cdmihai cdmihai force-pushed the issue1294/assemblyInfo branch from e609bdc to 7129454 Compare November 21, 2016 21:46
Copy link
Contributor

@AndyGerlicher AndyGerlicher left a comment

Choose a reason for hiding this comment

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

All I can say about this is I'm glad you wrote tests for it. :)

Copy link
Member

@rainersigwald rainersigwald left a comment

Choose a reason for hiding this comment

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

Yet another "if I hadn't been there debugging this with you I would probably think there had to be a better way but I can't see one" situation.

Copy link
Member

Choose a reason for hiding this comment

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

I think this'd read clearer if it was just "\\r\\n" : "\\n" instead of the concatenation.

@cdmihai cdmihai merged commit 5480712 into dotnet:xplat Nov 22, 2016
@cdmihai cdmihai deleted the issue1294/assemblyInfo branch February 2, 2017 01:37
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.

4 participants