-
Notifications
You must be signed in to change notification settings - Fork 872
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
Add nullable support #5166
Comments
Hi @kenkendk If this is still open, I'd like to pick this up |
@bigboybamo Awesome! I am not aware of any progress on the issue, so you can start where you like :) Starting "bottom-up" will make less back-trips to fix things once the libraries are nullable enabled. |
Hi @kenkendk I started with the Utility Library. I have the type annotations in a few files but I want to be sure I am on the right track (and its what you want) Can I open a small PR so you look at the changes? Then, maybe if its fine I can continue |
After the update to a modern C# version, the libraries should be updated one-by-one to support
nullable
notation.To avoid long-standing PRs, there should be one or two libraries in each PR.
Simply add:
To the
.csproj
file and fix the type annotations to correctly support null-able.The text was updated successfully, but these errors were encountered: