Skip to content

feat: trigger website SDK version sync on dakera-py release (DAK-5974)#124

Merged
ferhimedamine merged 1 commit into
mainfrom
dak-5974/trigger-website-sync
Jun 11, 2026
Merged

feat: trigger website SDK version sync on dakera-py release (DAK-5974)#124
ferhimedamine merged 1 commit into
mainfrom
dak-5974/trigger-website-sync

Conversation

@ferhimedamine

Copy link
Copy Markdown
Contributor

Summary

Adds a trigger-website-sync job to the PyPI publish workflow that fires a repository_dispatch event to the website repo after each successful Python SDK release.

How it works:

  • Fires after the publish job completes
  • Uses DOCS_CHECKOUT_TOKEN if present in this repo's secrets
  • Gracefully no-ops with a warning if the secret is absent
  • Passes the release tag as client_payload.version

Setup required:
Add DOCS_CHECKOUT_TOKEN secret to dakera-ai/dakera-py repo — the same PAT already available in dakera-ai/dakera. This enables the dispatch.

Companion PRs:

Test plan

  • Add DOCS_CHECKOUT_TOKEN to dakera-py repo secrets
  • Merge this PR
  • On next SDK release: verify website PR is created with updated SDK version

🤖 Generated with Claude Code

@ferhimedamine ferhimedamine added the auto-merge Auto-merge when CI passes label Jun 11, 2026
@ferhimedamine

Copy link
Copy Markdown
Contributor Author

🤖 [Agent: CTO] Code Review: APPROVED

Classification: type:ci-infra — release workflow addition, no bench required.

Review notes:

  • Adds trigger-website-sync job to release.yml, fires after PyPI publish
  • continue-on-error: true — correct, website sync failure must not block SDK releases
  • Good: graceful empty-token check with actionable warning message
  • Minimal, focused change — 22 lines added

CI: ✅ All 5 checks green (Examples, integration, test 3.10, test 3.12, update_release_draft). Merging now.

@ferhimedamine ferhimedamine merged commit aceb234 into main Jun 11, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Auto-merge when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant