Skip to content

Conversation

@cloudn9
Copy link
Contributor

@cloudn9 cloudn9 commented Apr 20, 2020

Continuing list from #9744

@cloudn9 cloudn9 requested a review from Thraka as a code owner April 21, 2020 18:48
Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

I think in some cases the ... are necessary even though they don't pass the XML checker. They're more symbolic.

cloudn9 and others added 19 commits April 23, 2020 16:44
…ion.md

Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
…ion.md

Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Co-Authored-By: Genevieve Warren <gewarren@microsoft.com>
Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

@cloudn9 Thanks for fixing these! I added the ... back in a few places.

@cloudn9
Copy link
Contributor Author

cloudn9 commented Apr 24, 2020

@cloudn9 Thanks for fixing these! I added the ... back in a few places.

Oh thank you @gewarren :)

@gewarren gewarren merged commit 7c2b21a into dotnet:master Apr 24, 2020
@cloudn9 cloudn9 deleted the bugfix/cloudn9/other-xml-issues branch April 25, 2020 10:56
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.

4 participants