fix: gracefully handle missing DOCS_PREVIEW_DEPLOY_KEY secret - #127
Conversation
- Add token existence check before deployment - Skip deployment if secret is not configured - Post informative PR comment when deployment is skipped - Prevents workflow failure when secret is missing/invalid Fixes authentication failure in docs preview deployment workflow
There was a problem hiding this comment.
Sorry @JacobCoffee, you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
|
🚅 Deployed to the byte-pr-127 environment in byte
|
✅ Docs Preview Repository CreatedThe
Setup Complete✅ Repository created with proper structure Required Action: Configure Deploy TokenTo enable documentation preview deployments, you need to create a Personal Access Token: Step 1: Generate Token
Step 2: Add Token to Secrets
VerificationOnce the token is configured, you can verify the setup by:
ArchitectureThis follows the same pattern as Litestar's polyfactory-docs-preview:
The workflow will now gracefully handle missing tokens and post informative comments to PRs. |
Summary
Changes
Check if deploy token existsstep to validate secret presenceRoot Cause
The workflow was failing because
DOCS_PREVIEW_DEPLOY_KEYsecret is either:byte-docs-previewrepositoryTest Plan
Post-Merge Action Required
Repository administrator needs to configure
DOCS_PREVIEW_DEPLOY_KEYsecret:reposcopeDOCS_PREVIEW_DEPLOY_KEY🤖 Generated with Claude Code
Co-Authored-By: Claude noreply@anthropic.com