-
Notifications
You must be signed in to change notification settings - Fork 26
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
fix: model test utils serviceName bug #878
Conversation
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.
Looks good! Just one comment about the changelog
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.
I'd recommend removing the changelog, since this is just changing a test utility and not customer-facing.
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.
Will it still allow me to merge?
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.
Yep, you just have to add the no-changelog
label on GitHub (tab to the right)
…route53-custom-error-unmarshalling
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This reverts commit afb36c8.
Fixed an issue with utility. SDK ID for
route 53
requires space but service name can't have space, i.e.route53
Issue #
No Issue available (Will make one if required)
Description of changes
It solves the problem if the utility not working when it comes to route 53
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.