Skip to content

gitsync: fix subsequent checkouts of reference "refs/heads/main" and tags#289

Draft
srenatus wants to merge 2 commits intomainfrom
sr/fix-gitsync/maybe
Draft

gitsync: fix subsequent checkouts of reference "refs/heads/main" and tags#289
srenatus wants to merge 2 commits intomainfrom
sr/fix-gitsync/maybe

Conversation

@srenatus
Copy link
Contributor

@srenatus srenatus commented Mar 9, 2026

The bug was that reference: refs/heads/main would only check out once,
and not update the local repo and checkout after commits were pushed to
the remote.

🚧 So far, no test reproduces the original problem; so this isn't over yet. In the setup, the service was repeatedly run in one-shot-mode, which is not in those tests yet. ⚠️

@srenatus srenatus force-pushed the sr/fix-gitsync/maybe branch from 23e2be9 to c8466c7 Compare March 9, 2026 19:33
@srenatus srenatus changed the title gitsync: remove SingleBranch: true gitsync: fix subsequent checkouts of reference "refs/heads/main" Mar 9, 2026
@srenatus srenatus force-pushed the sr/fix-gitsync/maybe branch 2 times, most recently from 2a52b54 to c53ee0f Compare March 9, 2026 19:38
@srenatus srenatus changed the title gitsync: fix subsequent checkouts of reference "refs/heads/main" gitsync: fix subsequent checkouts of reference "refs/heads/main" and tags Mar 9, 2026
@srenatus srenatus marked this pull request as ready for review March 9, 2026 19:44
The bug was that `reference: refs/heads/main` would only check out once,
and not update the local repo and checkout after commits were pushed to
the remote.

Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
@srenatus srenatus force-pushed the sr/fix-gitsync/maybe branch from c53ee0f to 82d2456 Compare March 9, 2026 20:17
@srenatus srenatus marked this pull request as draft March 9, 2026 21:07
Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
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.

1 participant