8000 AWS::ECR::Repository by philtay · Pull Request #395 · cloudtools/troposphere · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

AWS::ECR::Repository #395

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

Merged
merged 2 commits into from
Mar 1, 2016
Merged

AWS::ECR::Repository #395

merged 2 commits into from
Mar 1, 2016

Conversation

philtay
Copy link
Contributor
@philtay philtay commented Feb 26, 2016

@jeanphix
Copy link
Contributor

@philtay maybe you could squash these two commits.

@phobologic
Copy link
Member

Looks good @philtay - could you add an example so that we get tests? The one here should be good: http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ecr-repository.html#d0e45175

@philtay
Copy link
Contributor Author
philtay commented Mar 1, 2016

@phobologic Currently Amazon ECR is not present in awacs. A testable example like the one you linked can be added only after the inclusion of ECR in awacs. Moreover, given the fact that troposphere installs awacs from PyPI, a new version of awacs should be released before adding the example. I'm going to open a PR on awacs for this. In the meantime, considering that this is a very simple resource (15 lines of Python the entire file), I think that it can be safely merged. As soon as a new release of awacs will be out I will add the test.

@phobologic
Copy link
Member

Ok - I honestly was fine with just a dict implementation for an example/tests, but I can see waiting for an awacs integrated example as well. Thanks @philtay

phobologic added a commit that referenced this pull request Mar 1, 2016
@phobologic phobologic merged commit a62b13e into cloudtools:master Mar 1, 2016
@philtay philtay mentioned this pull request Mar 5, 2016
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
0