refactor(event-handler): remove kwargs from AppSync exception constructor#7699
Conversation
|
Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need. |
pull-request-size
bot
added
the
size/XS
|
T90REAL
changed the title
**kwargs from UnauthorizedException class.**kwargs from UnauthorizedException class.
T90REAL
changed the title
**kwargs from UnauthorizedException class.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #7699 +/- ##
========================================
Coverage 96.52% 96.52%
========================================
Files 275 275
Lines 13117 13117
Branches 986 986
========================================
Hits 12661 12661
Misses 353 353
Partials 103 103 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
dreamorosi
left a comment
dreamorosi
left a comment
There was a problem hiding this comment.
Hi @T90REAL, thank you for sending the PR.
Well done 🚢 !
dreamorosi
changed the title
dreamorosi
changed the title
|
Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience! |
2 participants
Issue number: closes #7693
Summary
Remove
**kwargsfrom the signature to avoid confusing customers.Changes
Delete
**kwargsfrom argument and doc.User experience
User will not be confused by the signatures.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.