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

Possible 'System.NullReferenceException' #1413

Closed

mokarchi wants to merge 2 commits into dotnet:main from mokarchi:main

mokarchi commented Feb 23, 2024

use conditional access


          Possible 'System.NullReferenceException'

80dc79b

josefpihrt previously approved these changes

View reviewed changes


          Merge branch 'main' into main

8d9d1de

josefpihrt dismissed their stale review

February 26, 2024 11:55

I took a look at it again and the value must be present here, otherwise it's ok to throw.

Collaborator

josefpihrt commented Feb 26, 2024

I took a look at it again and the value must be present here, otherwise it's ok to throw. So I'm going to close this PR.

@mokarchi Feel free to make another contribution, you can take a look at the list of issues, maybe you find something interesting here.

mokarchi closed this by deleting the head repository

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment