Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Fix issue where we were adding a parameter null check after it was of any use #66868

Merged
merged 1 commit into from
Feb 14, 2023
Merged

Fix issue where we were adding a parameter null check after it was of any use #66868

merged 1 commit into from
Feb 14, 2023

Conversation

CyrusNajmabadi
Copy link
Member

Fixes #38093

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner February 14, 2023 01:35
@CyrusNajmabadi CyrusNajmabadi merged commit ba6ebc0 into dotnet:main Feb 14, 2023
@ghost ghost added this to the Next milestone Feb 14, 2023
@CyrusNajmabadi CyrusNajmabadi deleted the addParamCheck branch February 14, 2023 04:14
@allisonchou allisonchou mentioned this pull request Feb 15, 2023
Closed
@RikkiGibson RikkiGibson modified the milestones: Next, 17.6 P2 Mar 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dibarbet dibarbet dibarbet approved these changes

@Cosifne Cosifne Cosifne approved these changes

Assignees
No one assigned
Labels
Area-IDE
Projects
None yet
Milestone
17.6 P2
Development

Successfully merging this pull request may close these issues.

"Add null check" adds meaningless null check
4 participants
@CyrusNajmabadi @dibarbet @Cosifne @RikkiGibson