Skip to content

Conversation

evanchooly
Copy link
Member

merge helper scripts to one file and parameterize it to reduce clutter and repetition

fixes #5037

@gsmet gsmet requested a review from patriot1burke November 18, 2019 16:42
@evanchooly evanchooly marked this pull request as ready for review November 20, 2019 14:30
Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

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

That looks good to me, but I'll wait until someone with more experience in AWS to approve ;)

@geoand
Copy link
Contributor

geoand commented Nov 20, 2019

I am grossly underqualified to review this since I don't know much of anything about amazon lambda :)

@evanchooly
Copy link
Member Author

it's mostly just the textual changes (not strictly amazon-related) that I wanted a newbie's eye on. And the two of you reviewed the last round so i figured I'd give it another shot. @oztimpower has verified the template changes, fwiw.

@geoand
Copy link
Contributor

geoand commented Nov 20, 2019

Sure, I'll go over the text, no problem

@gsmet
Copy link
Member

gsmet commented Nov 20, 2019

@evanchooly can you squash everything once you're done? Thanks!

@evanchooly
Copy link
Member Author

@gsmet for sure.

merge helper scripts to one file and parameterize it to reduce clutter and repetition
Copy link
Contributor

@geoand geoand left a comment

Choose a reason for hiding this comment

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

Docs look good!

@evanchooly evanchooly merged commit 9f05a1c into quarkusio:master Nov 21, 2019
@evanchooly evanchooly deleted the lambda-sam branch November 21, 2019 18:58
@gsmet gsmet added this to the 1.1.0 milestone Nov 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Amazon Lambda SAM CLI integration

4 participants