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.

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

Enable nullable reference types in Oakton project #102

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ArturDorochowicz
Copy link
Contributor

@ArturDorochowicz ArturDorochowicz commented Nov 9, 2024

Enable nullable reference types in Oakton project. This enables nullability warnings for the project (a number of them are reported now). Also, it will default newly added files to be nullable-enabled.

Prior nullable-oblivious types are preserved by adding #nullable disable annotations to nearly all files (apart from two that had #nullable enable pragmas, some files in Internal.Conversions that used nullable references in signatures, and some files where the pragma is clearly redundant).

Skipped two files that had `#nullable enable` at the top.

This change allows to preserve the original meaning of declarations w.r.t. nullability.
Some files already used nullable reference types. They don't need disable annotations pragma.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant