Skip to content

Offer completion for #nullable enable|disable#31171

Merged
jcouv merged 5 commits intodotnet:dev16.0-preview2from
jcouv:nullable-completion
Nov 15, 2018
Merged

Offer completion for #nullable enable|disable#31171
jcouv merged 5 commits intodotnet:dev16.0-preview2from
jcouv:nullable-completion

Conversation

@jcouv
Copy link
Member

@jcouv jcouv commented Nov 14, 2018

This is mostly an IDE fix, but also includes a minor fix to compiler APIs.

#nullable
image

#nullable enable|disable
image

Addresses part of #31130

@jcouv jcouv added this to the 16.0.P2 milestone Nov 14, 2018
@jcouv jcouv self-assigned this Nov 14, 2018
@jcouv jcouv requested review from a team as code owners November 14, 2018 05:55
@jcouv
Copy link
Member Author

jcouv commented Nov 14, 2018

Thanks!

@jcouv jcouv force-pushed the nullable-completion branch from 52aed75 to c20ff50 Compare November 15, 2018 00:22
@jcouv
Copy link
Member Author

jcouv commented Nov 15, 2018

@jinujoseph for preview2 approval. Thanks

Copy link
Contributor

@AlekseyTs AlekseyTs left a comment

Choose a reason for hiding this comment

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

Code under Compilers LGTM (iteration 5)

Copy link
Contributor

@cston cston left a comment

Choose a reason for hiding this comment

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

Compiler changes LGTM.

@jcouv jcouv merged commit 7c04f24 into dotnet:dev16.0-preview2 Nov 15, 2018
@jcouv jcouv deleted the nullable-completion branch November 15, 2018 21:14
@jcouv jcouv removed the Blocked label Nov 15, 2018
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.

6 participants