Address Feedback

- missing period
- removed cloud.gov specific instructions
This commit is contained in:
ctro 2016-03-23 10:18:12 -06:00
parent 04cd63c42b
commit 175787810a

View file

@ -28,7 +28,7 @@ jobs:
``` ```
### Options ### Options
The `options` parameter is synonymous with the options that `aws cli` accepts for `sync`. Please see [S3 Sync Options](http://docs.aws.amazon.com/cli/latest/reference/s3/sync.html#options) and pay special attention to the [Use of Exclude and Include Filters](http://docs.aws.amazon.com/cli/latest/reference/s3/index.html#use-of-exclude-and-include-filters) The `options` parameter is synonymous with the options that `aws cli` accepts for `sync`. Please see [S3 Sync Options](http://docs.aws.amazon.com/cli/latest/reference/s3/sync.html#options) and pay special attention to the [Use of Exclude and Include Filters](http://docs.aws.amazon.com/cli/latest/reference/s3/index.html#use-of-exclude-and-include-filters).
Given the following directory `test` we can upload _only_ the `results` subdirectory by using the `options` JSON below. Given the following directory `test` we can upload _only_ the `results` subdirectory by using the `options` JSON below.
@ -65,7 +65,6 @@ Requires [Docker](https://www.docker.com/).
1. Run `cp config.example.json config.json`. 1. Run `cp config.example.json config.json`.
1. Modify `config.json`. 1. Modify `config.json`.
* See [the instructions for getting your AWS credentials](http://docs.aws.amazon.com/cli/latest/userguide/cli-chap-getting-set-up.html#cli-signup). * See [the instructions for getting your AWS credentials](http://docs.aws.amazon.com/cli/latest/userguide/cli-chap-getting-set-up.html#cli-signup).
* Recommended local S3 setup is via `cloud.gov`'s [s3 service and a Static app](https://docs.cloud.gov/apps/s3/).
* Exclude the `s3://` prefix/protocol for `bucket`. * Exclude the `s3://` prefix/protocol for `bucket`.
1. Run `./test/out </full/path/to/dir>`. 1. Run `./test/out </full/path/to/dir>`.