dotnet / dotnet/templating

Do not validate nullable annotations when targeting net472

Open
#9,494 0 comments 0 reactions 0 assignees View on GitHub
triage:transfer-to-sdk
Dominant language
No language data
Stars
1.7k
Forks
399
Avg merge
3d 5h
Merged PRs (30d)
39

Description

The code base currently uses `!` suppression to avoid nullable warnings.
Rather than littering the code with `!` the nullability analysis should be disabled for net472 targets in projects that multi-target.

Alternatively, calls to APIs like `string.IsNullOrEmpty` that are annotated on net10 but not on net472 should be changed to null-annotated equivalents.

The editorconfig suppresses `IDE0370: Suppression is unnecessary` to allow unnecessary `!` when not targeting net472.
This should be removed.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.