Closed tobias-tengler closed 3 weeks ago
It seems all right 👌
No new problems were found according to the checks applied
💡 Qodana analysis was run in the pull request mode: only the changed files were checked ☁️ View the detailed Qodana report
Would it be more appropriate to allow the default scope be customizable similar to DefaultMaxAge
, or should that be deferred to a future enhancement?
Yes, this can also be done, but it wouldn't address the underlying issue. I'm going to add it later.
1 new problem were found
Inspection name | Severity | Problems |
---|---|---|
Redundant 'object.ToString()' call |
🔶 Warning | 1 |
💡 Qodana analysis was run in the pull request mode: only the changed files were checked ☁️ View the detailed Qodana report
Attention: Patch coverage is 90.00000%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 72.08%. Comparing base (
723733e
) to head (d0c9fb1
). Report is 3 commits behind head on main.:exclamation: Current head d0c9fb1 differs from pull request most recent head 9d576ad. Consider uploading reports for the commit 9d576ad to get more accurate results
Files | Patch % | Lines |
---|---|---|
...olate/Caching/src/Caching/CacheControlAttribute.cs | 0.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Fixes #7055