Skip to content

Conversation

andrew-dixon
Copy link
Contributor

A couple of minor grammatical corrections I noticed while reading the page.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

A couple of minor grammatical corrections I noticed while reading the page.
Copy link
Contributor

@brettstack brettstack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@brettstack brettstack changed the base branch from master to develop June 26, 2018 22:38

The [AWS SAM CLI](https://github.com/awslabs/aws-sam-cli) builds on top of the SAM specification by providing a single tool to manage the packaging and deployment of serverless applications. Once the `sam` tool is installed, the application deployment process occurs in two phases.

First, use `sam` to upload the binary to S3 and reference it in a newly created `packaged.yaml` CloudFormation configuration.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First, use sam package

$ diff template.yaml packaged.yaml
```

Lastly, use `sam` again to deploy the template through a CloudFormation stack named `Test`.
Copy link
Contributor

@dhruvsood dhruvsood Jun 28, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change "Lastly, use sam again" to "Second, use sam package..."


### Deployment

The [AWS SAM CLI](https://github.com/awslabs/aws-sam-cli) builds on top of the SAM specification by providing a single tool to manage the packaging and deployment of serverless applications. Once the `sam` tool is installed, the application deployment process occurs in two phases.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change this to:
The AWS SAM CLI lets you locally build, test, and debug serverless applications defined by AWS SAM templates. You can also use ‘sam cli’ to deploy your application. Install sam cli, and then deploy your application in two steps:

Copy link
Contributor

@dhruvsood dhruvsood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this, @andrew-dixon. I've added a couple of minor comments.

@brettstack
Copy link
Contributor

@andrew-dixon I merged develop in as there was a conflict from another PR I just merged into develop. Could you take a look at the comments from @sooddhruv and we'll get this merged in. Thanks!

@andrew-dixon
Copy link
Contributor Author

andrew-dixon commented Jun 29, 2018

@brettstack Not sure if I'm missing something but I don't see any comments and the things above that say "outdated" are to a different file anyway.

@dhruvsood
Copy link
Contributor

@andrew-dixon yeah, those comments were for examples/2016-10-31/schedule/README.md. I've submitted a PR to address those.

Your changes to HOWTO.md look good. Thank you for your contribution!

Copy link
Contributor

@dhruvsood dhruvsood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@brettstack brettstack merged commit ca0fe86 into aws:develop Jun 29, 2018
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