Skip to content

Added jq requirement to examples readme #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jun 3, 2020

Conversation

waterskier2007
Copy link
Contributor

No description provided.

@swift-server-bot
Copy link

Can one of the admins verify this patch?

3 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@waterskier2007
Copy link
Contributor Author

attempts to address #105

@fabianfett fabianfett requested a review from tomerd June 2, 2020 13:16
@tomerd
Copy link
Contributor

tomerd commented Jun 2, 2020

thanks @waterskier2007

@@ -19,6 +19,8 @@ Steps to deploy this sample to AWS Lambda using the AWS CLI:
```

Note: This script assumes you have AWS CLI installed and credentials setup in `~/.aws/credentials`.

Note: You must also have the [jq](https://stedolan.github.io/jq/download/) command line tool installed.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is also required by the SAM example below, so maybe move this ~L6?

Copy link
Contributor

@tomerd tomerd 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 adding this, question about the right location for this

@tomerd tomerd linked an issue Jun 2, 2020 that may be closed by this pull request
@waterskier2007
Copy link
Contributor Author

confused by the line recommended in the comment. Are you suggesting to relocate to L6?

@tomerd
Copy link
Contributor

tomerd commented Jun 2, 2020

hi @waterskier2007, yea add it before the sections for CLI and SAM since its required by both

@waterskier2007
Copy link
Contributor Author

@tomerd hopefully this resolves the issue

@fabianfett
Copy link
Member

@swift-server-bot test this please

@waterskier2007 waterskier2007 requested a review from tomerd June 3, 2020 14:43
@tomerd tomerd merged commit 5e546a1 into swift-server:master Jun 3, 2020
@waterskier2007 waterskier2007 deleted the patch-1 branch June 3, 2020 18:04
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.

jq is required for the build/deploy of examples
4 participants