Skip to content

Conversation

@chengfang
Copy link
Collaborator

@chengfang chengfang commented Sep 10, 2025

Current go.mod references a fixed version of registry-scanner library. Often we need to modify registry-scanner library API and use that in image-updater in the same commit or pull request. In local environment, we can manually uncomment the replace line, but CI checks will fail. The usual fix to this problem is to split the changes into 2 pull requests: the 1st for the changes in registry-scanner, and 2nd for the changes in image-updater referencing the newly updated registry-scanner API. Some pull requests (e.g., #1231) were experiencing this issue.

This PR enables immediate use of any changes in registry-scanner across image-updater codebase.

@codecov-commenter
Copy link

codecov-commenter commented Sep 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.77%. Comparing base (5f7bb29) to head (f7cdcad).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1234   +/-   ##
=======================================
  Coverage   63.77%   63.77%           
=======================================
  Files          23       23           
  Lines        3213     3213           
=======================================
  Hits         2049     2049           
  Misses       1054     1054           
  Partials      110      110           

☔ 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.

@chengfang chengfang marked this pull request as draft September 10, 2025 14:34
…nd activate the its replace clause for development

Signed-off-by: Cheng Fang <[email protected]>
@chengfang chengfang marked this pull request as ready for review September 10, 2025 14:51
@chengfang chengfang requested a review from dkarpele as a code owner September 10, 2025 14:51
@dkarpele dkarpele merged commit 9daecb3 into argoproj-labs:master Sep 10, 2025
11 checks passed
dkarpele pushed a commit to dkarpele/argocd-image-updater that referenced this pull request Sep 16, 2025
…nd activate the its replace clause for development (argoproj-labs#1234)

Signed-off-by: Cheng Fang <[email protected]>

(cherry picked from commit 9daecb3)
Signed-off-by: Denis Karpelevich <[email protected]>
Sign up for free to 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