Skip to content

Comments

docs(sdk): migrate handling-external-contributor-pr playbook#16502

Merged
stephanie-anderson merged 7 commits intomasterfrom
dev-docs/playbooks/handling-external-contributor-pr
Feb 23, 2026
Merged

docs(sdk): migrate handling-external-contributor-pr playbook#16502
stephanie-anderson merged 7 commits intomasterfrom
dev-docs/playbooks/handling-external-contributor-pr

Conversation

@dingsdax
Copy link
Contributor

@dingsdax dingsdax commented Feb 21, 2026

DESCRIBE YOUR PR

Migrates the handling-external-contributor-pr playbook. Changes applied:

  • Added spec frontmatter with version 1.0.0, candidate status
  • Added SpecRfcAlert, SpecMeta, and SpecChangelog components
  • Added Overview section with related resources
  • Applied RFC 2119 keywords (REQUIRES, MUST) to all requirements
  • Created reusable saved reply includes with full CopyableCard components
  • Saved replies now display with copy-to-clipboard functionality in both locations
  • Updated saved-replies document to use same includes for DRY principle
  • Expanded Referenced Standards section with 3 standards and descriptions
  • Emphasized timely, respectful responses and "closing fast is kind" principle
  • Verified all referenced standard sections exist

The playbook guides SDK maintainers through triaging external PRs with consistent, respectful communication.

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

Co-Authored-By: Claude noreply@anthropic.com

@vercel
Copy link

vercel bot commented Feb 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment Feb 23, 2026 11:25am
sentry-docs Ready Ready Preview, Comment Feb 23, 2026 11:25am

Request Review

…format

- Added spec frontmatter with version tracking
- Added SpecRfcAlert, SpecMeta, and SpecChangelog components
- Added Overview section with related resources
- Applied RFC 2119 keywords to requirements
- Created reusable saved reply includes (partials) for DRY principle
- Inlined saved replies in playbook using includes
- Updated saved-replies document to use same includes
- Expanded Referenced Standards section with 3 standards
- Emphasized timely, respectful responses
- Verified all referenced standard sections exist

Co-Authored-By: Claude <noreply@anthropic.com>
@dingsdax dingsdax force-pushed the dev-docs/playbooks/handling-external-contributor-pr branch from 3fdb0ad to 7d1d037 Compare February 21, 2026 21:04
- Updated includes to contain complete CopyableCard with title
- Removed blockquote wrapper in playbook to show full component
- Copy-to-clipboard functionality now works in both locations
- Saved replies look identical in playbook and saved-replies doc

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Instead of using Include components which caused double-nesting issues, the saved-replies page now contains the CopyableCard components directly, and playbooks link to them.

Changes:
- Saved-replies index: Inline CopyableCard content instead of includes
- Handling-external-contributor-pr playbook: Link to saved replies instead of including them
- Remove unused include files

This makes saved-replies the single source of truth without duplication.

Co-Authored-By: Claude <noreply@anthropic.com>
@dingsdax dingsdax force-pushed the dev-docs/playbooks/handling-external-contributor-pr branch from f3a33a9 to ca881e5 Compare February 23, 2026 11:12
@stephanie-anderson stephanie-anderson merged commit cf3e032 into master Feb 23, 2026
15 checks passed
@stephanie-anderson stephanie-anderson deleted the dev-docs/playbooks/handling-external-contributor-pr branch February 23, 2026 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants