-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat(QAB-164): Update s3 steps to use aws golang sdk v2 #104
Conversation
rt PATH=/Users/zc01167/sdk/go1.18.2/bin:$PATH clear
Codecov Report
@@ Coverage Diff @@
## master #104 +/- ##
==========================================
- Coverage 62.35% 62.03% -0.33%
==========================================
Files 47 47
Lines 2925 2924 -1
==========================================
- Hits 1824 1814 -10
- Misses 1039 1046 +7
- Partials 62 64 +2
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.
Have a look at my comments/questions?
Kudos, SonarCloud Quality Gate passed! |
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.
LGTM
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Please provide enough information so that others can review your pull request:
Explain the details for making this change. What existing problem does the pull request solve?
Documentation
Update Readme docs if needed.
Test plan (required)
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.
Code formatting
Closing issues
Put
closes #XXXX
in your comment to auto-close the issue that your PR fixes (if such).