Skip to content

Always invalidate matching packages during sync#31

Merged
cottsay merged 1 commit intomainfrom
cottsay/sync-always-remove
Feb 10, 2026
Merged

Always invalidate matching packages during sync#31
cottsay merged 1 commit intomainfrom
cottsay/sync-always-remove

Conversation

@cottsay
Copy link
Member

@cottsay cottsay commented Jan 12, 2026

Right now, invalidation only occurs in the target repository when there was at least one package matching sync criteria in the upstream repository. This behavior is unexpected given that the names of the matching packages have no bearing on what packages are invalidated during a typical sync operation.

Closes #5

Requires #29, so submitted as draft until that is merged.

@cottsay cottsay self-assigned this Jan 12, 2026
@cottsay cottsay added the bug Something isn't working label Jan 12, 2026
@cottsay
Copy link
Member Author

cottsay commented Jan 12, 2026

Ha, well #29 also has a prerequisite change, so CI on this PR will fail until one or both of those upstream changes are merged.

@cottsay cottsay linked an issue Jan 16, 2026 that may be closed by this pull request
Base automatically changed from cottsay/python-sync to main February 10, 2026 21:02
Right now, invalidation only occurs in the target repository when there
was at least one package matching sync criteria in the upstream
repository. This behavior is unexpected given that the names of the
matching packages have no bearing on what packages are invalidated
during a typical sync operation.

Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay force-pushed the cottsay/sync-always-remove branch from 48d6b2b to 806da04 Compare February 10, 2026 21:02
@cottsay cottsay marked this pull request as ready for review February 10, 2026 21:03
@codecov-commenter
Copy link

codecov-commenter commented Feb 10, 2026

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 48.97%. Comparing base (c6f7f54) to head (806da04).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/createrepo-agent/command.c 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   48.86%   48.97%   +0.11%     
==========================================
  Files          20       20              
  Lines        2642     2638       -4     
  Branches      538      538              
==========================================
+ Hits         1291     1292       +1     
+ Misses       1046     1039       -7     
- Partials      305      307       +2     

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

@cottsay cottsay merged commit 895f0ac into main Feb 10, 2026
3 checks passed
@cottsay cottsay deleted the cottsay/sync-always-remove branch February 10, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sync operation doesn't remove pattern when source is empty

3 participants