Skip to content

🐛 Update remaining v1alpha1 references with v1 - #1452

Merged
perdasilva merged 1 commit into
operator-framework:mainfrom
m1kola:fix_v1alpha1_refs
Nov 13, 2024
Merged

🐛 Update remaining v1alpha1 references with v1#1452
perdasilva merged 1 commit into
operator-framework:mainfrom
m1kola:fix_v1alpha1_refs

Conversation

@m1kola

Copy link
Copy Markdown
Member

Description

There are a few remaining references to v1alpha1 that need to be updated to use v1.

Ref:

Reviewer Checklist

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

@openshift-ciopenshift-ciBot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 13, 2024
@netlify

netlifyBot commented Nov 13, 2024

Copy link
Copy Markdown

Deploy Preview for olmv1 ready!

NameLink
🔨 Latest commit932501a
🔍 Latest deploy loghttps://app.netlify.com/sites/olmv1/deploys/6734a3f047ed190008fdb316
😎 Deploy Previewhttps://deploy-preview-1452--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.

@m1kola

Copy link
Copy Markdown
MemberAuthor

Blocked by #1451

@m1kola
m1kola marked this pull request as ready for review November 13, 2024 11:12
@m1kola
m1kola requested a review from a team as a code ownerNovember 13, 2024 11:12
@openshift-ciopenshift-ciBot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 13, 2024

@camilamacedo86camilamacedo86 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@openshift-ciopenshift-ciBot added the lgtm Indicates that a PR is ready to be merged. label Nov 13, 2024
There are a few remaining references to v1alpha1 that need to be
updated to use v1.
Ref:
* operator-framework#1228
Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com>
@openshift-ciopenshift-ciBot removed the lgtm Indicates that a PR is ready to be merged. label Nov 13, 2024
@openshift-ci

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@m1kola

Copy link
Copy Markdown
MemberAuthor

Rebased on top of #1451 so it can pass upgrade-e2e.

@perdasilva
perdasilva added this pull request to the merge queue Nov 13, 2024
Merged via the queue into operator-framework:main with commit 102f29fNov 13, 2024
@m1kola
m1kola deleted the fix_v1alpha1_refs branch November 13, 2024 14:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@m1kola@perdasilva@camilamacedo86