Skip to content
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

Add example on how to use S3 events for lambda functions written in Go using sam #737

Merged
merged 8 commits into from
Jan 3, 2019

Conversation

shuza
Copy link
Contributor

@shuza shuza commented Dec 27, 2018

Description of changes: This PR shows how to use aws sam to develop application that will trigger lambda function written in Go on S3 events like image upload.

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

@codecov-io
Copy link

codecov-io commented Dec 28, 2018

Codecov Report

Merging #737 into release/v1.10.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v1.10.0     #737   +/-   ##
================================================
  Coverage            94.18%   94.18%           
================================================
  Files                   67       67           
  Lines                 2685     2685           
  Branches               478      478           
================================================
  Hits                  2529     2529           
  Misses                  80       80           
  Partials                76       76

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffb3e35...0f72c9a. Read the comment docs.

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.

Thanks for this contribution, it looks good! Thank you for modifying the README and giving instructions on how to build and use this example, I will follow these steps and report if there are any issues. Just need to make that one template fix and this should be good to go.

examples/2016-10-31/image_resize_golang/template.yaml Outdated Show resolved Hide resolved
@shuza
Copy link
Contributor Author

shuza commented Jan 1, 2019

@keetonian thanks for reviewing my pull request and the advise.
template issue has been fixed.

keetonian
keetonian previously approved these changes Jan 3, 2019
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.

Ran the example and it works, thanks! Fixed a few small template issues and removed the go binary file, thank you for this contribution!

@keetonian keetonian changed the base branch from master to release/v1.10.0 January 3, 2019 00:24
paoptu023
paoptu023 previously approved these changes Jan 3, 2019
examples/2016-10-31/image_resize_golang/README.md Outdated Show resolved Hide resolved
@keetonian keetonian dismissed stale reviews from paoptu023 and themself via 0f72c9a January 3, 2019 17:43
@keetonian keetonian merged commit 2542388 into aws:release/v1.10.0 Jan 3, 2019
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.

5 participants