-
Notifications
You must be signed in to change notification settings - Fork 25
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 the move command in retrievePreviousBuild to fix bug in windows #378
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #378 +/- ##
=========================================
Coverage 86.11% 86.11%
Complexity 29 29
=========================================
Files 82 82
Lines 216 216
Branches 12 12
=========================================
Hits 186 186
Misses 22 22
Partials 8 8 ☔ View full report in Codecov by Sentry. |
build.gradle
Outdated
@@ -121,7 +121,7 @@ jacocoTestReport { | |||
} | |||
} | |||
|
|||
String version = '6.2.0' | |||
String version = '6.2.1' |
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.
Can you not bump the version? I can take care of it in my upcoming PR. Thanks! Need to bump minor version
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.
When are you planning to bump this version? This is fixing the distribution build so we might want to bump it soon.
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.
By EOD. We are having too many tags for 1 fixes. Trying to club 3 changes together.
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 changed it back. Please prioritize the version bump so that we could update our build Jenkinsfiles with new tag version. Thanks.
Signed-off-by: Zelin Hao <[email protected]>
#378) Signed-off-by: Zelin Hao <[email protected]> (cherry picked from commit d7dd07c) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Update the move command in retrievePreviousBuild to fix bug in windows
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.