Skip to content

Commit 848af67

Browse files
jamierpondclaude
andauthored
fix(ci): add sync-docs to release workflows, pass GITHUB_TOKEN for web build (#109)
The Consolidate docs PR moved topic files to docs/topics/ with a sync-docs make target, but both release workflows were missed. GoReleaser builds fail because go:embed can't find cli/internal/docs/topics/. Web tests fail because the blog build needs GITHUB_TOKEN to fetch content from the external madea.blog repo. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6a3f0a5 commit 848af67

3 files changed

Lines changed: 8 additions & 0 deletions

File tree

.github/workflows/next-release.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,9 @@ jobs:
4040
git tag -f "$VERSION"
4141
git push origin "$VERSION"
4242
43+
- name: Sync docs for embed
44+
run: make sync-docs
45+
4346
- name: Run GoReleaser
4447
uses: goreleaser/goreleaser-action@v6
4548
with:

.github/workflows/release.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,9 @@ jobs:
2727
exit 1
2828
fi
2929
30+
- name: Sync docs for embed
31+
run: make sync-docs
32+
3033
- uses: goreleaser/goreleaser-action@v6
3134
with:
3235
version: "~> v2"

.github/workflows/web-tests.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@ jobs:
4242

4343
- name: Build web
4444
run: pnpm build:web
45+
env:
46+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
4547

4648
- name: Run unit tests
4749
run: pnpm test

0 commit comments

Comments
 (0)