docs(oss): the notify example must use gh api -f, not -F - #269
Conversation
gh api -F reads a value that starts with @ from a file, so '@prisma/composer-cli' fails to parse before any request is made. A product repository that copies the example verbatim never sends the dispatch (prisma/composer#295 did exactly that). -f sends the literal string. Signed-off-by: willbot <w.a.madden+machine@gmail.com> Signed-off-by: Will Madden <madden@prisma.io> Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. Summary by CodeRabbit
WalkthroughThe release automation documentation updates the publish workflow notification command. The command now uses Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to This documentation-only change has no established merge risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
commit: |
docs/oss/release-automation.mdshows product repositories how to send theproduct-publisheddispatch withgh api ... -F 'client_payload[package]=@prisma/composer-cli'.-Freads a value that starts with@from a file, so that line fails withopen prisma/composer-cli: no such file or directorybefore any request is made. prisma/composer#295 copied the example verbatim and its review bot caught it.-fsends the literal string.Two characters changed; no code.
🤖 Generated with Claude Code