-
Notifications
You must be signed in to change notification settings - Fork 407
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
docs(metrics): Correct code examples in markdown #316
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #316 +/- ##
========================================
Coverage 99.88% 99.88%
========================================
Files 91 91
Lines 3339 3341 +2
Branches 162 163 +1
========================================
+ Hits 3335 3337 +2
Misses 2 2
Partials 2 2
Continue to review full report at Codecov.
|
Sorry one more @heitorlessa . I am picking them up as i try to copy and paste from the docs. |
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.
Thanks ;) I had left no MetricUnit to make it easier for people coming from bare SDK or EMF library.
But makes sense to keep using our enum as that's a good feature
@heitorlessa cool, which ever code examples you would prefer people to copy from. |
…tools-python into develop * 'develop' of https://github.com/awslabs/aws-lambda-powertools-python: feat(logging): Include exception_name (#320) chore: remove gatsby mention as migrated completed refactor(parameters): Consistently reference env (#319) docs(metrics): remove minimum dimensions docs: Correct code examples (#317) docs(metrics): Correct code examples in markdown (#316) fix(idempotency): TypeError when calling is_missing_idempotency_key with an int (#315) docs(metrics): Corrections to the code examples (#314) fix(idempotency): Correctly handle save_inprogress errors (#313)
Description of changes:
Correct the metrics code examples to use
MetricUnit.Count
when it was imported otherwiseremove when not used in code sample
Checklist