S3 bucket Cloud Formation Syntax

beaknit / cform

SublimeText plugin for CloudFormation

Apache License 2.0

103 stars 34 forks source link

Closed neilramsay closed 9 years ago

neilramsay commented 9 years ago

CloudFormation S3 Bucket and S3 Bucket Policy from:

beaknit commented 9 years ago

PR's 10 & 11 look right to me but I'm not in front of my machine to test them. I'll take a look tomorrow

neilramsay commented 9 years ago

PR11 is currently missing "AWS::Route53::HealthCheck". I will add that in later.

beaknit commented 9 years ago

Could you split this into two snippets?

neilramsay commented 9 years ago

Yes, I have been doing two snippets with the first containing both, and the second containing the *policy only. Is that sufficient?

beaknit commented 9 years ago

Since either can be used without the other, I'd rather keep them separate entirely.

neilramsay commented 9 years ago

I think it is ready now. Please review.

beaknit commented 9 years ago

LGTM