Skip to content

Conversation

hsbakshi
Copy link
Contributor

This canary doesn't need access to DynamoDB.

Issue #, if available:

Description of changes:
Avoid unnecessary dependency on DynamoDB.

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

This canary doesn't need access to DynamoDB.
@keetonian keetonian changed the base branch from master to develop August 1, 2018 21:57
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.

👍 Thank you, @hsbakshi!

Copy link
Contributor

@keetonian keetonian left a comment

Choose a reason for hiding this comment

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

Thank you for catching this! This sample application should not have a dependency on DynamoDB

@keetonian keetonian merged commit 6a70a01 into aws:develop Aug 2, 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.

4 participants