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

use existing response.vary property to set vary header #2346

Merged
merged 1 commit into from
May 29, 2017
Merged

use existing response.vary property to set vary header #2346

merged 1 commit into from
May 29, 2017

Conversation

davidism
Copy link
Member

closes #2345

@davidism davidism self-assigned this May 29, 2017
@codecov
Copy link

codecov bot commented May 29, 2017

Codecov Report

Merging #2346 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2346      +/-   ##
==========================================
- Coverage   92.86%   92.85%   -0.02%     
==========================================
  Files          46       46              
  Lines        6423     6414       -9     
  Branches      503      501       -2     
==========================================
- Hits         5965     5956       -9     
  Misses        336      336              
  Partials      122      122
Impacted Files Coverage Δ
flask/helpers.py 82.77% <ø> (-0.41%) ⬇️
flask/sessions.py 95.7% <100%> (-0.03%) ⬇️
tests/test_basic.py 96.5% <100%> (ø) ⬆️

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 5797416...b8eba0a. Read the comment docs.

@davidism davidism merged commit b7eca82 into pallets:master May 29, 2017
@davidism davidism deleted the 2345-vary branch May 29, 2017 17:16
@davidism davidism modified the milestone: 1.0 Jul 31, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use existing response.vary property to set vary header
2 participants