-
-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Update NullabilityInfoContext.cs.pp with bug fix from MS repo #7
Conversation
@SimonCropp When using this source-only repo, does the code get injected for all frameworks, or only pre-.NET 6 frameworks? If the latter, is there any way to make this work in a .NET 6 environment, so that the fixed code is used rather than the buggy .NET 6 code? |
thanks for this. i will deploy it now.
only pre-.NET 6 frameworks
not currently. TBH it is no a scenario that i considered. although it is obviously valid. thoughts on how it could be achieved? perhaps via compilation constant? |
or i could deploy a new nuget that has all the types in a diff namespace |
😕 What happens with the MS code? |
It is slated for .NET 7 |
it should have been a nuget that is shipped out of band |
See: