Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add nullability support to use local function #36458

Merged
merged 1 commit into from
Jun 17, 2019
Merged

Add nullability support to use local function #36458

merged 1 commit into from
Jun 17, 2019

Conversation

jasonmalinowski
Copy link
Member

Fixes #30322

@jasonmalinowski jasonmalinowski requested a review from ryzngard June 14, 2019 17:51
@jasonmalinowski jasonmalinowski requested a review from a team as a code owner June 14, 2019 17:51
@jasonmalinowski jasonmalinowski self-assigned this Jun 14, 2019
@jasonmalinowski
Copy link
Member Author

Tagging @jinujoseph for a M2 approval; trivial fix.

@jinujoseph jinujoseph added this to the 16.2.P4 milestone Jun 14, 2019
@jasonmalinowski jasonmalinowski force-pushed the fix-use-local-function branch from 84d6167 to 54f4e4a Compare June 14, 2019 20:09
@jasonmalinowski jasonmalinowski merged commit 62c9a74 into dotnet:master Jun 17, 2019
@jasonmalinowski jasonmalinowski deleted the fix-use-local-function branch June 17, 2019 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CyrusNajmabadi CyrusNajmabadi CyrusNajmabadi approved these changes

@jinujoseph jinujoseph jinujoseph approved these changes

@ryzngard ryzngard Awaiting requested review from ryzngard

Assignees

@jasonmalinowski jasonmalinowski

Labels
Approved to merge Area-IDE
Projects
None yet
Milestone
16.2.P4
Development

Successfully merging this pull request may close these issues.

UseLocalFunction isn't aware of nullability
3 participants
@jasonmalinowski @CyrusNajmabadi @jinujoseph