-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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 golang version #21014
update golang version #21014
Conversation
to golang v1.23.2 Signed-off-by: wang yan <wangyan@vmware.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #21014 +/- ##
===========================================
+ Coverage 45.36% 66.17% +20.80%
===========================================
Files 244 1048 +804
Lines 13333 114469 +101136
Branches 2719 2856 +137
===========================================
+ Hits 6049 75752 +69703
- Misses 6983 34574 +27591
- Partials 301 4143 +3842
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Signed-off-by: wang yan <wangyan@vmware.com>
Signed-off-by: wang yan <wangyan@vmware.com>
Signed-off-by: wang yan <wangyan@vmware.com>
Signed-off-by: wang yan <wangyan@vmware.com>
Signed-off-by: wang yan <wangyan@vmware.com>
Signed-off-by: wang yan <wangyan@vmware.com>
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
@@ -28,7 +28,7 @@ jobs: | |||
- name: Set up Go 1.21 | |||
uses: actions/setup-go@v5 | |||
with: | |||
go-version: 1.22.3 | |||
go-version: 1.23.2 |
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.
name should be "Set up Go 1.23"
to golang v1.23.2
Thank you for contributing to Harbor!
Comprehensive Summary of your change
Issue being fixed
Fixes #(issue)
Please indicate you've done the following: