Skip to content

chore(cli): re-pin go-flashduty to merged main (follow-up to #52)#54

Merged
ysyneu merged 1 commit into
feat/ai-srefrom
chore/cli-repin-goflashduty-merged
Jun 16, 2026
Merged

chore(cli): re-pin go-flashduty to merged main (follow-up to #52)#54
ysyneu merged 1 commit into
feat/ai-srefrom
chore/cli-repin-goflashduty-merged

Conversation

@ysyneu

@ysyneu ysyneu commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Follow-up hygiene fix to #52.

#52 was merged at its pre-re-pin commit, so feat/ai-sre ended up pinning the go-flashduty#11 PR-branch commit (da82c4) instead of the squash-merged main commit (54d4fc1). That orphaned commit currently resolves only because the closed PR keeps it alive — fragile.

This re-pins go.mod to go-flashduty main. Generated zz_generated_* is unchanged (identical spec content), so the diff is go.mod/go.sum only. go build + go test ./internal/... pass; cligen re-run produces no drift.

#52 merged at the pre-re-pin commit, so feat/ai-sre pinned the go-flashduty#11
PR-branch commit (da82c4) rather than the squash-merged main commit. Re-pin to
go-flashduty main (54d4fc1) so the dependency tracks main, not an orphaned
PR-branch commit kept alive only by the closed PR. Generated output unchanged
(identical spec); build + unit tests pass.
@ysyneu ysyneu merged commit e93ef96 into feat/ai-sre Jun 16, 2026
12 checks passed
@ysyneu ysyneu deleted the chore/cli-repin-goflashduty-merged branch June 16, 2026 05:34
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