search

repository-service-tuf / repository-service-tuf-worker

Repository Service for TUF: Worker
MIT License
8 stars 15 forks source link

Fix force md update with custom delegations #504

Closed MVrachev closed 1 month ago

MVrachev commented 2 months ago

Fixes #503

Fix a bug which happened when we wanted to bump targets and a custom delegated role at the same time which caused a bump of the target role and all custom delegated roles at the same time.

Note: Merge #501 first.

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (714a29d) to head (56457ac). Report is 39 commits behind head on main.

:exclamation: Current head 56457ac differs from pull request most recent head 2a04a7a. Consider uploading reports for the commit 2a04a7a to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #504 +/- ## ========================================== Coverage 100.00% 100.00% ========================================== Files 15 15 Lines 1071 1204 +133 ========================================== + Hits 1071 1204 +133 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

MVrachev commented 1 month ago

Ready for review @kairoaraujo. I rebased it on top of the main branch and tested it manually.

MVrachev commented 1 month ago

@kairoaraujo I addressed your comment in my latest commit. Ready for review.