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

Update not create #484

Merged
merged 5 commits into from
Jun 3, 2016
Merged

Update not create #484

merged 5 commits into from
Jun 3, 2016

Conversation

chrisgilmerproj
Copy link
Contributor

Working with @markfalk discovered its easier to use this script if it has an update option. This change includes these updates while maintaining backwards compatibility:

  • Can now use -u or --update to trigger a cfn update action instead of a create action
  • Fixes pep8 trailing whitespace
  • Fixes pep8 line length to be < 80 chars

@chrisgilmerproj
Copy link
Contributor Author

Friendly ping - Hoping to get a review on this PR.

@chrisgilmerproj
Copy link
Contributor Author

Merged this with master, I think its ready to review.

@markpeek markpeek merged commit 9265bfa into cloudtools:master Jun 3, 2016
@markpeek
Copy link
Member

markpeek commented Jun 3, 2016

Thanks!

@chrisgilmerproj
Copy link
Contributor Author

Thank you!

amosshapira pushed a commit to amosshapira/troposphere that referenced this pull request Oct 24, 2016
* Add ability to update instead of always running create
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