Enable nullability in shared test sources - #38792
Open
AndriySvyryd wants to merge 1 commit into
Open
Conversation
This was referenced Aug 12, 2026
cincuranet
approved these changes
Aug 12, 2026
There was a problem hiding this comment.
Pull request overview
This PR removes file-level #nullable enable directives from shared source files so they inherit the nullable context from the consuming projects, completing nullable analysis coverage across test/benchmark builds as part of #24427.
Changes:
- Removed top-of-file
#nullable enabledirectives from shared utility sources linked into multiple projects. - Standardized nullable context handling by relying on project-level nullable configuration instead of per-file directives.
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/Shared/SharedTypeExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/PropertyInfoExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/OrderedDictionary.ValueCollection.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/OrderedDictionary.KeyCollection.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/OrderedDictionary.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/NonCapturingLazyInitializer.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/Multigraph.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/MemberInfoExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/IDictionaryDebugView.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/HashHelpers.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/ExpressionVisitorExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/ExpressionExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/EnumerableExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/DisposableExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/DictionaryExtensions.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/CodeAnnotations.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/Check.cs | Drops file-level nullable directive to rely on project nullable settings. |
| src/Shared/BidirectionalDictionary.cs | Drops file-level nullable directive to rely on project nullable settings. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
stack merge was automatically disabled
August 12, 2026 21:22
Pull Request is not mergeable
stack merge was automatically disabled
August 12, 2026 21:23
Pull Request is not mergeable
AndriySvyryd
force-pushed
the
issue24427-nrt-5-shared
branch
from
August 12, 2026 21:25
372a620 to
db11b32
Compare
stack merge was automatically disabled
August 12, 2026 22:03
Pull Request is not mergeable
AndriySvyryd
force-pushed
the
issue24427-nrt-5-shared
branch
from
August 12, 2026 22:03
db11b32 to
a2fdc6f
Compare
AndriySvyryd
force-pushed
the
issue24427-nrt-5-shared
branch
from
August 12, 2026 22:34
a2fdc6f to
6553399
Compare
- Annotate shared source files linked into nullable-enabled test projects - Complete nullable analysis coverage across the solution Fixes #24427
AndriySvyryd
force-pushed
the
issue24427-nrt-5-shared
branch
from
August 13, 2026 00:40
6553399 to
5bf4797
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part 5 of 5 for #24427. Based on #38791.
Summary
Stack
Validation
EFCore.slnxwith 0 warnings and 0 errors.100.main, and that project is excluded from CI.Fixes #24427