Skip to content

Conversation

sambattalio
Copy link
Contributor

Issue #, if available:
#1063
Description of changes:

Updated the policy template as described in the issue, and related tests.

Let me know if there is anything else I need to do!

Description of how you validated changes:

Ensured that the tests worked, and tested by running bin/sam-translate.py on a test template yaml and manually checking the transformed template to ensure that the additional actions were present.

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.

@codecov-io
Copy link

Codecov Report

Merging #1072 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1072   +/-   ##
========================================
  Coverage    94.81%   94.81%           
========================================
  Files           69       69           
  Lines         3276     3276           
  Branches       639      639           
========================================
  Hits          3106     3106           
  Misses          89       89           
  Partials        81       81

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 527c190...0323425. Read the comment docs.

@praneetap praneetap merged commit f61e707 into aws:develop Aug 9, 2019
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