-
-
Notifications
You must be signed in to change notification settings - Fork 744
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use CacheControlScope.Public per default #7058
Conversation
Qodana for .NETIt 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 Contact Qodana teamContact us at qodana-support@jetbrains.com
|
Would it be more appropriate to allow the default scope be customizable similar to |
Yes, this can also be done, but it wouldn't address the underlying issue. I'm going to add it later. |
0f484f9
to
d0c9fb1
Compare
Qodana for .NET1 new problem were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7058 +/- ##
==========================================
- Coverage 77.27% 72.08% -5.20%
==========================================
Files 2632 2621 -11
Lines 131349 130776 -573
==========================================
- Hits 101505 94265 -7240
- Misses 29844 36511 +6667
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Fixes #7055