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

Bump up to go1.22 #7666

Merged
merged 1 commit into from
Apr 15, 2024
Merged

Conversation

reasonerjt
Copy link
Contributor

@reasonerjt reasonerjt commented Apr 12, 2024

Bump up the golang version to v1.22

It also updates controller-gen to v0.14.0 to fix an issue under new
version of go.
More details see golang/go#65637

It's known that it causes decrease to the coverage on codecov due to the change in go1.22 to calculate coverage

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

@github-actions github-actions bot added Dependencies Pull requests that update a dependency file has-changelog labels Apr 12, 2024
@reasonerjt reasonerjt marked this pull request as draft April 12, 2024 09:07
@reasonerjt reasonerjt force-pushed the bumpup-to-go1.22-new branch 2 times, most recently from 351fab3 to dcd9382 Compare April 12, 2024 09:43
Copy link

codecov bot commented Apr 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.86%. Comparing base (c888f51) to head (1b3fe95).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7666      +/-   ##
==========================================
- Coverage   61.29%   56.86%   -4.43%     
==========================================
  Files         276      343      +67     
  Lines       31490    28391    -3099     
==========================================
- Hits        19301    16144    -3157     
- Misses      10830    10849      +19     
- Partials     1359     1398      +39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@reasonerjt reasonerjt changed the title [WIP] Bumpup to go1.22 Bump up to go1.22 Apr 12, 2024
@reasonerjt reasonerjt marked this pull request as ready for review April 12, 2024 10:31
@reasonerjt reasonerjt force-pushed the bumpup-to-go1.22-new branch 3 times, most recently from e275545 to f7613cd Compare April 12, 2024 12:16
hack/test.sh Outdated Show resolved Hide resolved
hack/test.sh Outdated Show resolved Hide resolved
Copy link
Contributor

@mateusoliveira43 mateusoliveira43 left a comment

Choose a reason for hiding this comment

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

left comments

@blackpiglet
Copy link
Contributor

The UT is impacted. Please double-check the change mentioned by @mateusoliveira43.

@reasonerjt reasonerjt marked this pull request as draft April 14, 2024 09:46
@reasonerjt
Copy link
Contributor Author

@blackpiglet

The UT is impacted. Please double-check the change mentioned by @mateusoliveira43.

The coverage may have to be impacted, b/c the go1.22 stopped skipping the packages that do not include _test files, I find it also changed in other details, for example, it stopped taking into some inline comments as covered.

@reasonerjt reasonerjt changed the title Bump up to go1.22 [WIP] Bump up to go1.22 Apr 14, 2024
This commit bumps up the golang for building and testing velero to v1.22

It also updates controller-gen to v0.14.0 to fix an issue under new
versino of go.
More details see golang/go#65637

Signed-off-by: Daniel Jiang <jiangd@vmware.com>
@reasonerjt reasonerjt marked this pull request as ready for review April 15, 2024 07:07
@reasonerjt reasonerjt changed the title [WIP] Bump up to go1.22 Bump up to go1.22 Apr 15, 2024
@ywk253100 ywk253100 merged commit f3295cc into vmware-tanzu:main Apr 15, 2024
69 of 70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Pull requests that update a dependency file has-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants