diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2026-01-27 21:55:34 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2026-01-27 21:55:34 +0000 |
| commit | efc3da477d4edb9d1334718e3e20d197ba711468 (patch) | |
| tree | a00daf545c8bd5dc723676cbe07d2b20d5a9080f /docs/archive/2026-01-relay-ngit-dev-migration/scripts | |
| parent | 6d920cae2704016869500889a92b358d845b69e1 (diff) | |
fix: pass actually fetched OIDs to process_newly_available_git_data
Previously, sync_identifier_from_url passed all needed OIDs to
process_newly_available_git_data, not just the OIDs that were
successfully fetched. This caused incorrect logging (new_oids_count
would show all needed OIDs, not just fetched ones).
While this didn't break functionality (the actual processing uses
can_apply_state which checks the repository on disk), it made
debugging confusing.
Changes:
- Rename oids_fetched to fetched_oids and change type from usize to Vec<String>
- Return Vec<String> from match arms instead of counts
- Pass fetched_oids (not needed_oids) to process_newly_available_git_data
- Return fetched_oids.len() at the end
This ensures logging accurately reflects which OIDs were actually
fetched from the remote.
Diffstat (limited to 'docs/archive/2026-01-relay-ngit-dev-migration/scripts')
0 files changed, 0 insertions, 0 deletions