Fix nullability mismatch between ref and src in Microsoft.Extensions.Caching.Abstractions

[maxkoshevoi]

I missed a spot in #64018

Somehow, PostEvictionDelegate and CacheExtensions were annotated differently in ref and scr.
@eerhardt CI might need some improvement to catch those issues (if ref projects are not removed)

[dotnet-issue-labeler]

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

[eerhardt]

@eerhardt CI might need some improvement to catch those issues (if ref projects are not removed)

It's on the backlog, but not yet funded. We would need:

• [API Compat] Add rule to make sure nullable annotations are compatible sdk#18681
• [API Compat] Add MSBuild task to run API Compat standalone sdk#18677

To be fixed first. For now, be sure to run GenerateReferenceSource in your PRs that change the nullable annotations.

FYI @ericstj.

[eerhardt]

Thanks for your quick response on this @maxkoshevoi!

LGTM

[maxkoshevoi]

be sure to run GenerateReferenceSource in your PRs that change the nullable annotations.

But running dotnet msbuild /t:GenerateReferenceAssemblySource doesn't update nullable annotations for me.. (at least it didn't a month ago)

[ghost]

Tagging subscribers to this area: @dotnet/area-extensions-caching
See info in area-owners.md if you want to be subscribed.

Issue Details

---

I missed a spot in #64018

Somehow, PostEvictionDelegate and CacheExtensions were annotated differently in ref and scr.
@eerhardt CI might need some improvement to catch those issues (if ref projects are not removed)

Author:	maxkoshevoi
Assignees:	eerhardt
Labels:	area-Extensions-Caching, community-contribution
Milestone:	-

[ericstj]

But running dotnet msbuild /t:GenerateReferenceAssemblySource doesn't update nullable annotations for me.. (at least it didn't a month ago)

This should if you have a built the library already. If you have a project where this isn't working let us know and I'll take a quick look. -- Actually file an issue so we don't lose it -- area-infrastucture-libraries.