aptly-dev / aptly

aptly - Debian repository management tool
https://www.aptly.info/
MIT License
2.56k stars 374 forks source link

Skip loading reflists when listing published repos #1274

Closed neolynx closed 5 months ago

neolynx commented 5 months ago

Replaces #1227

Requirements

All new code should be covered with tests, documentation should be updated. CI should pass.

Description of the Change

Loading the reflists to list publish information takes a significant amount of time, even though the resulting info doesn't actually use any of the reflists.

Checklist

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 73.68421% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 74.79%. Comparing base (27013c0) to head (1325f7b).

Files Patch % Lines
deb/publish.go 70.58% 4 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1274 +/- ## ======================================= Coverage 74.78% 74.79% ======================================= Files 144 144 Lines 16246 16256 +10 ======================================= + Hits 12150 12158 +8 - Misses 3154 3156 +2 Partials 942 942 ```

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