-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
New service: TimestreamWrite service client with single database resource #15444
New service: TimestreamWrite service client with single database resource #15444
Conversation
Thank you for your contribution! 🚀 Please note that typically Go dependency changes are handled in this repository by Renovate Bot or the maintainers. This is to prevent pull request merge conflicts and further delay reviews of contributions. Remove any changes to the Additional details:
|
I removed the mod/sum files, but I think the tests fail because of its removed. Also, this is my first OSS PR, and my first ever Go code I wrote. |
Hey @yardensachs, Thanks you for your contribution. |
@DrFaust92 Will do |
e9d9fd7
to
5ed2159
Compare
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Closes #15421
Release note for CHANGELOG:
Output from acceptance testing: