Skip to content

Conversation

@ShreyaGangishetty
Copy link

Issue #, if available:

Description of changes:
cherry pick #1517 into release branch

Description of how you validated changes:

Checklist:

  • Write/update tests
  • make pr passes
  • Update documentation
  • Verify transformed template deploys and application functions as expected
  • Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ShreyaGangishetty ShreyaGangishetty changed the base branch from develop to release/v1.22.0 March 16, 2020 20:18
@codecov-io
Copy link

codecov-io commented Mar 16, 2020

Codecov Report

Merging #1520 into release/v1.22.0 will decrease coverage by 0.01%.
The diff coverage is 85.71%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           release/v1.22.0    #1520      +/-   ##
===================================================
- Coverage            94.11%   94.09%   -0.02%     
===================================================
  Files                   78       78              
  Lines                 4942     4949       +7     
  Branches               996      999       +3     
===================================================
+ Hits                  4651     4657       +6     
  Misses                 135      135              
- Partials               156      157       +1
Impacted Files Coverage Δ
samtranslator/model/eventsources/push.py 90.87% <100%> (+0.03%) ⬆️
samtranslator/open_api/open_api.py 91.6% <80%> (-0.24%) ⬇️

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 dd1c3d3...7dc8a20. Read the comment docs.

@ShreyaGangishetty ShreyaGangishetty merged commit 19ed4a0 into aws:release/v1.22.0 Mar 16, 2020
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.

4 participants