Skip to content

Conversation

@ShreyaGangishetty
Copy link

…(#1264)

Issue #, if available:
Merged #1264 into master instead of develop.

Description of changes:
This PR is to merge #1264 into develop and I will submit another PR to revert #1264 from master

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.

@keetonian
Copy link
Contributor

Note: requires a merge commit rather than a squash+merge.

@keetonian
Copy link
Contributor

Can you fix the failing tests? Looks like there are some policy template differences you need to pick up on develop.

@ShreyaGangishetty
Copy link
Author

@keetonian I will check and fix failing tests. Thank you

@ShreyaGangishetty ShreyaGangishetty mentioned this pull request Dec 6, 2019
5 tasks
@ShreyaGangishetty
Copy link
Author

@keetonian Please review #1306 . Closing this in favor of #1306

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.

3 participants