-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update Python AWS Lambda integration docs. #8246
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there! I have some wording suggestions. Also, it would be great if the text in a few of the screenshots were bigger b/c it's hard to see at the moment. Let me know if you have any questions or would like to talk about any of my suggestions!
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/platforms/python/common/integrations/aws-lambda/manual-layer/index.mdx
Outdated
Show resolved
Hide resolved
src/platforms/python/common/integrations/aws-lambda/manual-layer/index.mdx
Outdated
Show resolved
Hide resolved
src/platforms/python/common/integrations/aws-lambda/manual-layer/index.mdx
Outdated
Show resolved
Hide resolved
src/platforms/python/common/integrations/aws-lambda/manual-layer/index.mdx
Outdated
Show resolved
Hide resolved
src/platforms/python/common/integrations/aws-lambda/manual-layer/index.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few tiny little tweaks, but this is looking great. Thanks for making the changes and updating the images!
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/docs/product/integrations/cloud-monitoring/aws-lambda/index.mdx
Outdated
Show resolved
Hide resolved
src/platforms/python/common/integrations/aws-lambda/manual-instrumentation/index.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Liza Mock <[email protected]>
Added Python 3.10 and 3.11 to the `SUPPORTED_RUNTIMES` in the AWS Lambda integration. Related docs PR: getsentry/sentry-docs#8246
Bring all the information in the documentation for the AWS Lambda integration with Python up to date:
https://sentry-docs-git-antonpirker-python-aws-lambda.sentry.dev/product/integrations/cloud-monitoring/aws-lambda/
https://sentry-docs-git-antonpirker-python-aws-lambda.sentry.dev/platforms/python/integrations/aws-lambda/
(I ignored the "AWS Lambda with Container Image" pages because this feature is kind of abandoned right now and needs some more work in the code before we can update the documentation)