Skip to content

Conversation

perdasilva
Copy link
Contributor

No description provided.

Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.48%. Comparing base (08ab7fb) to head (60e7542).
Report is 67 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (08ab7fb) and HEAD (60e7542). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (08ab7fb) HEAD (60e7542)
2 1
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #416      +/-   ##
==========================================
- Coverage   85.06%   78.48%   -6.59%     
==========================================
  Files          19       31      +12     
  Lines        1346     2477    +1131     
==========================================
+ Hits         1145     1944     +799     
- Misses        125      444     +319     
- Partials       76       89      +13     

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

@perdasilva perdasilva marked this pull request as draft January 13, 2025 10:17
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 13, 2025
@perdasilva
Copy link
Contributor Author

/hold until controller-runtime is updated to 0.32

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 13, 2025
@perdasilva perdasilva marked this pull request as ready for review January 13, 2025 14:07
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 13, 2025
@openshift-ci openshift-ci bot requested a review from varshaprasad96 January 13, 2025 14:07
@perdasilva
Copy link
Contributor Author

/unhold

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 13, 2025
Copy link
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 14, 2025
@perdasilva perdasilva enabled auto-merge January 14, 2025 13:30
@perdasilva perdasilva added this pull request to the merge queue Jan 14, 2025
Merged via the queue into operator-framework:main with commit c923823 Jan 14, 2025
6 checks passed
@perdasilva perdasilva deleted the bump/k8s branch January 14, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/deps area/sdk lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants