Skip to content
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

Improve caching directive to be able to be conditional to resource #155

Merged
merged 2 commits into from
Apr 21, 2022

Conversation

jvanecek
Copy link
Member

No description provided.

@jvanecek jvanecek requested a review from gcotelli April 20, 2022 22:13
@codecov
Copy link

codecov bot commented Apr 20, 2022

Codecov Report

Merging #155 (34cf93d) into release-candidate (683620f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@                Coverage Diff                 @@
##           release-candidate     #155   +/-   ##
==================================================
  Coverage              99.77%   99.77%           
==================================================
  Files                    164      164           
  Lines                  12842    12857   +15     
==================================================
+ Hits                   12813    12828   +15     
  Misses                    29       29           
Impacted Files Coverage Δ
...thAmericanCurrenciesRESTfulControllerTest.class.st 100.00% <100.00%> (ø)
.../SouthAmericanCurrenciesRESTfulController.class.st 100.00% <100.00%> (ø)
...-Model-Tests/CachingDirectivesBuilderTest.class.st 100.00% <100.00%> (ø)
...e/Stargate-Model/CachingDirectivesBuilder.class.st 100.00% <100.00%> (ø)
...STfulControllerRespondCreatedEntityPolicy.class.st 100.00% <100.00%> (ø)
...rgate-Model/RESTfulRequestHandlerBehavior.class.st 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 683620f...34cf93d. Read the comment docs.

@gcotelli gcotelli merged commit 5878e15 into release-candidate Apr 21, 2022
@gcotelli gcotelli deleted the cache-control-conditional-to-resource branch April 21, 2022 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants