Skip to content

fix(ci): force-fetch upstream tags in Upstream Sync#30

Merged
bdruth merged 1 commit into
flexfrom
fix/upstream-sync-force-tags
May 20, 2026
Merged

fix(ci): force-fetch upstream tags in Upstream Sync#30
bdruth merged 1 commit into
flexfrom
fix/upstream-sync-force-tags

Conversation

@bdruth
Copy link
Copy Markdown

@bdruth bdruth commented May 20, 2026

Summary

Second fix after #29 unblocked auth. The next workflow run got past checkout but failed at `git fetch upstream --tags --prune` with:

```
! [rejected] v0.6.40 -> v0.6.40 (would clobber existing tag)
```

This fork carries legacy tags from when it was created. At least `v0.6.40` points to a different commit than upstream's `v0.6.40`, so `git fetch --tags` refuses the overwrite by default.

We always want upstream's tag values when resolving the sync target, so `--force` is correct. `--prune` is kept but is not `--prune-tags`, so any flex-only tags remain untouched.

Test plan

  • Merge
  • Run `Upstream Sync` via `workflow_dispatch`
  • Confirm the fetch step succeeds and the workflow either exits cleanly ("Up to date" — flex is already on v0.9.5, the latest upstream release) or opens a PR if a newer tag has shipped

After fixing the auth issue (#29), the workflow failed at the next
step with:

  ! [rejected] v0.6.40 -> v0.6.40 (would clobber existing tag)

This fork carries legacy tags inherited from when it was created.
At least one of them (v0.6.40) points to a different commit than
upstream's tag of the same name, so `git fetch --tags` refuses the
overwrite by default.

We always want upstream's tag values when resolving the sync target,
so `--force` is the right behavior. `--prune` is kept but does NOT
prune tags (no --prune-tags), so any flex-only tags are safe.
@bdruth bdruth merged commit ca7284c into flex May 20, 2026
@bdruth bdruth deleted the fix/upstream-sync-force-tags branch May 20, 2026 15:36
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