Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

Tagging subscribers to this area: @eerhardt, @maryamariyan, @michaelgsharp
See info in area-owners.md if you want to be subscribed.

    public async Task OnceExpiredIsSetToTrueItRemainsTrue()

Any reason why this test wasn't turned into a Theory?

Refers to: src/libraries/Microsoft.Extensions.Caching.Memory/tests/CacheEntryScopeExpirationTests.cs:513 in 4684c51. [](commit_id = 4684c51, deletion_comment = False)

Any reason why this test wasn't turned into a Theory?

There is just one entry being created in this test (no dependencies to other linked entries) and it just verifies whether setting entry as expired is thread-safe (not related to tracking)

Failures are unrelated. Timing out due to dotnet/core-eng#14346

arm64 improvements: dotnet/perf-autofiling-issues#1537

To be honest I've not merged it because I wanted to check if something in ASP.NET repo is going to get broken with this change. It has been a month since it was merged and nobody started screaming. I guess it just proves that it was good idea to introduce the breaking change and disable the tracking by default.

@adamsitnik someone finally screamed

@adamsitnik were you planning to create a breaking change document for this?

I just added dotnet/docs#31739

I just added dotnet/docs#31739

@maryamariyan big thanks for doing that!