Skip to content

Conversation

@sharwell
Copy link
Contributor

@sharwell sharwell commented Oct 4, 2019

🚧 Builds on #39053. After that one is merged, many files will drop out of this pull request.

Each commit was verified to compile successfully without new warnings, so reviewers do have the option of going commit-by-commit.

/// </summary>
string MetadataName { get; }

#nullable disable
Copy link
Contributor Author

Choose a reason for hiding this comment

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

📝 Skipping the Containing* properties made this pull request viable. I will not be addressing these properties in this pull request.

@sharwell sharwell force-pushed the symbol-annotations branch 2 times, most recently from 30377b3 to 6252bd2 Compare October 4, 2019 04:21
@sharwell sharwell force-pushed the symbol-annotations branch from bf34745 to f7cef51 Compare October 4, 2019 13:52
@sharwell sharwell added the Concept-Null Annotations The issue involves annotating an API for nullable reference types label Jan 3, 2020
@jcouv jcouv added the PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. label Jan 8, 2020
Base automatically changed from master to main March 3, 2021 23:52
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 PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants