Skip to content
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: upgrade golang to 1.20.1 #18370

Merged
merged 2 commits into from
Mar 17, 2023
Merged

Conversation

reasonerjt
Copy link
Contributor

Bump up to use golang v1.20.1

@reasonerjt reasonerjt requested a review from a team as a code owner March 17, 2023 06:48
@reasonerjt reasonerjt added the release-note/update Update or Fix label Mar 17, 2023
@codecov
Copy link

codecov bot commented Mar 17, 2023

Codecov Report

Merging #18370 (3236167) into main (339f5b1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #18370   +/-   ##
=======================================
  Coverage   67.43%   67.44%           
=======================================
  Files         974      974           
  Lines      106424   106424           
  Branches     2659     2659           
=======================================
+ Hits        71772    71776    +4     
+ Misses      30800    30792    -8     
- Partials     3852     3856    +4     
Flag Coverage Δ
unittests 67.44% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/lib/retry/retry.go 100.00% <100.00%> (ø)
src/pkg/permission/evaluator/rbac/casbin_match.go 89.13% <100.00%> (ø)

... and 6 files with indirect coverage changes

@reasonerjt reasonerjt assigned zyyw and wy65701436 and unassigned Vad1mo and OrlinVasilev Mar 17, 2023
Copy link
Contributor

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@reasonerjt reasonerjt merged commit bea8dec into goharbor:main Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/update Update or Fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants