fix: cache control arguments for cached_stampede#510
fix: cache control arguments for cached_stampede#510JeppeKlitgaard wants to merge 4 commits intoaio-libs:masterfrom
Conversation
`cached_stampede` now features the same cache control arguments as the `cached` decorator
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #510 +/- ##
===========================================
+ Coverage 56.72% 99.55% +42.82%
===========================================
Files 33 36 +3
Lines 2928 3564 +636
Branches 155 450 +295
===========================================
+ Hits 1661 3548 +1887
+ Misses 1243 12 -1231
+ Partials 24 4 -20
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 28 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
@JeppeKlitgaard thanks for this, mind adding tests? There should be ones for the |
|
As mentioned, tests please, otherwise we'll close this soon. |
cached_stampedenow features the same cache control argumentsas the
cacheddecorator