Skip to content

Annotate implementations of IPreprocessingSymbol#39201

Merged
sharwell merged 2 commits intodotnet:mainfrom
sharwell:symbol-annotations-16
Jun 27, 2022
Merged

Annotate implementations of IPreprocessingSymbol#39201
sharwell merged 2 commits intodotnet:mainfrom
sharwell:symbol-annotations-16

Conversation

@sharwell
Copy link
Copy Markdown
Contributor

No description provided.

@sharwell sharwell requested a review from a team as a code owner October 10, 2019 06:57
@jcouv jcouv added the PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. label Oct 22, 2019
@sharwell sharwell added the Concept-Null Annotations The issue involves annotating an API for nullable reference types label Jan 3, 2020
@sharwell sharwell marked this pull request as draft April 8, 2020 22:29
Base automatically changed from master to main March 3, 2021 23:52
@sharwell sharwell removed Blocked PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. labels Jun 7, 2022
@sharwell
Copy link
Copy Markdown
Contributor Author

sharwell commented Jun 7, 2022

@dotnet/roslyn-compiler this is now updated and ready for review

@sharwell sharwell marked this pull request as ready for review June 7, 2022 20:25
@jcouv jcouv self-assigned this Jun 8, 2022
Copy link
Copy Markdown
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)

@sharwell
Copy link
Copy Markdown
Contributor Author

sharwell commented Jun 8, 2022

@dotnet/roslyn-compiler for second review

@sharwell sharwell enabled auto-merge June 8, 2022 16:48
@jcouv jcouv disabled auto-merge June 8, 2022 16:49
@jcouv jcouv enabled auto-merge (squash) June 8, 2022 16:49
@jcouv
Copy link
Copy Markdown
Member

jcouv commented Jun 8, 2022

@sharwell I disabled and re-enabled auto-merge but with squash option selected. The compiler codebase squashes by convention. Thanks

@sharwell sharwell disabled auto-merge June 8, 2022 16:50
@sharwell sharwell merged commit 606a8b7 into dotnet:main Jun 27, 2022
@sharwell sharwell deleted the symbol-annotations-16 branch June 27, 2022 21:55
@ghost ghost added this to the Next milestone Jun 27, 2022
@RikkiGibson RikkiGibson modified the milestones: Next, 17.3 P3 Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Concept-Null Annotations The issue involves annotating an API for nullable reference types

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants