Skip to content

🌱 Bump github.com/containers/image/v5 from 5.33.1 to 5.34 - #1846

Closed
camilamacedo86 wants to merge 1 commit into
operator-framework:mainfrom
camilamacedo86:fix-controller-gen
Closed

🌱 Bump github.com/containers/image/v5 from 5.33.1 to 5.34#1846
camilamacedo86 wants to merge 1 commit into
operator-framework:mainfrom
camilamacedo86:fix-controller-gen

Conversation

@camilamacedo86

Copy link
Copy Markdown
Contributor

Description

Reviewer Checklist

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

@camilamacedo86
camilamacedo86 requested a review from a team as a code ownerMarch 6, 2025 12:40
@netlify

netlifyBot commented Mar 6, 2025

Copy link
Copy Markdown

Deploy Preview for olmv1 ready!

NameLink
🔨 Latest commit7978a84
🔍 Latest deploy loghttps://app.netlify.com/sites/olmv1/deploys/67c9a7f5bb66590008a49544
😎 Deploy Previewhttps://deploy-preview-1846--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.

@camilamacedo86
camilamacedo86force-pushed the fix-controller-gen branch 4 times, most recently from 34b3114 to e3cc2a2CompareMarch 6, 2025 13:29
@codecov

codecovBot commented Mar 6, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.39%. Comparing base (564f35a) to head (7978a84).
Report is 3 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #1846 +/- ##
=======================================
Coverage 68.39% 68.39% =======================================
Files 63 63 Lines 5136 5136 =======================================
Hits 3513 3513 Misses 1392 1392 Partials 231 231 
FlagCoverage Δ
e2e51.52% <ø> (-0.08%)⬇️
unit56.01% <ø> (ø)

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.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@camilamacedo86camilamacedo86 changed the title TEST🌱 Bump github.com/containers/image/v5 from 5.33.1 to 5.34Mar 6, 2025
Bumps [github.com/containers/image/v5](https://github.com/containers/image) from 5.33.1 to 5.34.0.
- [Release notes](https://github.com/containers/image/releases)
- [Commits](containers/image@v5.33.1...v5.34.0)
---
updated-dependencies:
- dependency-name: github.com/containers/image/v5
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
And downgrade :
replace github.com/proglottis/gpgme => github.com/proglottis/gpgme v0.1.3
Due: kubernetes-sigs/controller-tools#1158
Co-Author: Joe Lanford <joe.lanford@gmail.com>
@openshift-merge-robotopenshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 7, 2025
@openshift-merge-robot

Copy link
Copy Markdown

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@camilamacedo86

Copy link
Copy Markdown
ContributorAuthor

I am closing I think we decide to wait for controller-runtime fix.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-rebaseIndicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@camilamacedo86@openshift-merge-robot