Skip to content

🌱 Bump go.mod from go 1.21 to 1.22 - #859

Closed
perdasilva wants to merge 1 commit into
operator-framework:mainfrom
perdasilva:perdasilva/bump-go
Closed

🌱 Bump go.mod from go 1.21 to 1.22#859
perdasilva wants to merge 1 commit into
operator-framework:mainfrom
perdasilva:perdasilva/bump-go

Conversation

@perdasilva

Copy link
Copy Markdown
Contributor

Description

Updates to go 1.22

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

@perdasilva
perdasilva requested a review from a team as a code ownerMay 15, 2024 18:01
@perdasilvaperdasilva added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 15, 2024
@netlify

netlifyBot commented May 15, 2024

Copy link
Copy Markdown

Deploy Preview for olmv1 ready!

NameLink
🔨 Latest commita9e81fe
🔍 Latest deploy loghttps://app.netlify.com/sites/olmv1/deploys/6644fb2d742b2800079a6cf0
😎 Deploy Previewhttps://deploy-preview-859--olmv1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@perdasilvaperdasilva changed the title [hold] 🌱 Bump go.mod from go 1.21 to 1.22🌱 Bump go.mod from go 1.21 to 1.22May 15, 2024
@perdasilvaperdasilva removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 15, 2024
@perdasilva

Copy link
Copy Markdown
ContributorAuthor

/hold until community gives go ahead

@openshift-ciopenshift-ciBot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 15, 2024
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
@codecov

codecovBot commented May 15, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.18%. Comparing base (55c19a1) to head (a9e81fe).

Additional details and impacted files
@@ Coverage Diff @@## main #859 +/- ##
=======================================
Coverage 83.18% 83.18% =======================================
Files 15 15 Lines 874 874 =======================================
Hits 727 727 Misses 102 102 Partials 45 45 
FlagCoverage Δ
e2e62.24% <ø> (ø)
unit69.10% <ø> (-5.83%)⬇️

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

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

@tmshort

Copy link
Copy Markdown
Contributor

A lot of new components require go to update to 1.22...

@tmshort

Copy link
Copy Markdown
Contributor

See #909

@tmshort

Copy link
Copy Markdown
Contributor

Closed due to #909 being merged

@tmshorttmshort closed this Jun 5, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/holdIndicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@perdasilva@tmshort