Skip to content

Conversation

@brmur
Copy link
Contributor

@brmur brmur commented Jun 17, 2020

Cleaned up S3 and EC2 Javascript examples examples.
Added unit tests.
Commented and documented.

  • The submitter has added the default copyright notice to all files.

  • The submitter has added unit tests for all code paths, run all of them, and they all pass.

  • The submitter has run a linter on the code, and all of the submitter's team's minimum rules pass.

  • The submitter has added the team's minimum usage documentation to the code.

  • The submitter has had their Editor edit all comments and strings, and the submitter has incorporated any and all resulting edits.

  • The submitter has added all of the submitter's team's related API reporting metadata.

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

@brmur brmur requested a review from beckandros June 17, 2020 17:51
Copy link
Contributor

@beckandros beckandros left a comment

Choose a reason for hiding this comment

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

Edits for descriptions, comment strings.

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)


Inputs:
- REGION (into command line below)
- MY_KEY_PAIR (in command line below)
Copy link
Contributor

Choose a reason for hiding this comment

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

...into command line below

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)

CONDITIONS OF ANY KIND, either express or implied. See the License for the
specific language governing permissions and limitations under the License.

ABOUT THIS NODE.JS SAMPLE: This sample is part of the SDK for JavaScript Developer Guide topic at
Copy link
Contributor

Choose a reason for hiding this comment

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

...NODE.JS EXAMPLE: This example is... of the AWS SDK...
(also add a period at end of sentence)


Inputs:
- REGION (in command line input below)
- COMMAND (i.e. START or STOP) (in command line input below)
Copy link
Contributor

Choose a reason for hiding this comment

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

... (i.e.,...)
and could you combine these parens into one?


Inputs:
- REGION (in command line input below)
- STATE ('ON' or 'OFF') (in command line input below)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't combine these two parens into one?

@ghost ghost assigned brmur Jun 22, 2020
@brmur brmur removed the on review label Jun 24, 2020
@brmur brmur merged commit c0a76c6 into awsdocs:master Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants