-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 examples #2689
Update examples #2689
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2689 +/- ##
========================================
Coverage 95.29% 95.29%
========================================
Files 60 60
Lines 12252 12252
========================================
Hits 11676 11676
Misses 576 576 Continue to review full report at Codecov.
|
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.
A couple of quick questions, otherwise looks good.
@@ -1,1513 +1,5 @@ | |||
{ | |||
"version": "1.0", | |||
"examples": { | |||
"AddLayerVersionPermission": [ |
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.
This is deleting all of our Lambda examples. We may want to double check on this before merging.
@@ -62,42 +63,6 @@ | |||
"id": "to-retreive-attribute-values", | |||
"title": "To retrieve a list of attribute values" | |||
} | |||
], | |||
"GetProducts": [ |
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.
This is another one where we're completely deleting an operation example. Does this need to be upstreamed?
Sync examples with upstream source.
888c516
to
b4081b4
Compare
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.
🚢
No description provided.