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

Gradual Removal of #nullable disable Directives #45715

Open
v-wuzhai opened this issue Jan 6, 2025 · 0 comments
Open

Gradual Removal of #nullable disable Directives #45715

v-wuzhai opened this issue Jan 6, 2025 · 0 comments
Labels
Area-NetSDK untriaged Request triage from a team member

Comments

@v-wuzhai
Copy link
Member

v-wuzhai commented Jan 6, 2025

Describe the bug

With the recent merge of the PR that centralized the management of nullable reference types in the main Directory.Build.Props file, we have ensured consistent default behavior across the entire repository. This change also included adding #nullable disable at the top of each .cs file, excluding files in projects where nullable reference types are already enabled.

To further clean up and simplify our codebase, we will gradually remove all #nullable disable directives from the src and test directories. This will avoid duplicate configurations and ensure consistency.

This change is part of our ongoing effort to improve code quality and maintainability. By centralizing the management of nullable reference types, we can ensure a more streamlined and consistent development experience.

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-NetSDK untriaged Request triage from a team member labels Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-NetSDK untriaged Request triage from a team member
Projects
None yet
Development

No branches or pull requests

1 participant