Skip to content

Mark APIs as shipped.#43613

Merged
333fred merged 2 commits intodotnet:release/dev16.6from
333fred:mark-shipped
Apr 29, 2020
Merged

Mark APIs as shipped.#43613
333fred merged 2 commits intodotnet:release/dev16.6from
333fred:mark-shipped

Conversation

@333fred
Copy link
Member

@333fred 333fred commented Apr 23, 2020

Ran scripts/PublicAPI/mark-shipped.cmd. No code change.

@333fred 333fred requested review from a team as code owners April 23, 2020 18:57
@333fred 333fred requested a review from sharwell April 23, 2020 19:38
static Microsoft.CodeAnalysis.CSharp.SyntaxFactory.XmlTextNewLine(string text, bool continueXmlDocumentationComment) -> Microsoft.CodeAnalysis.SyntaxToken
static Microsoft.CodeAnalysis.CSharp.SyntaxFactory.XmlTextNewLine(string text) -> Microsoft.CodeAnalysis.SyntaxToken
Copy link
Contributor

Choose a reason for hiding this comment

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

📝 These moves are going to make it difficult to compare the changes between releases. Is there a way to avoid them?

Copy link
Member Author

Choose a reason for hiding this comment

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

We should take these now, and then we won't have to deal with them the next time we ship. This is because we're running a sort, and for some reason the current file is not sorted. If we don't do this, it's going to be much more of a pain.

@sharwell sharwell dismissed their stale review April 23, 2020 19:44

Issue was resolved

@333fred
Copy link
Member Author

333fred commented Apr 28, 2020

@dotnet/roslyn-compiler for review. I included the source-generator changes in this because it will give us a good way of rounding up exactly the breaking changes we make to the API as we evolve it for publishing.

Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (iteration 2)

@333fred 333fred merged commit 7c52141 into dotnet:release/dev16.6 Apr 29, 2020
@333fred 333fred deleted the mark-shipped branch April 29, 2020 23:13
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.

5 participants