Skip to content

chore: addressing IC feedback - #17708

Closed
stephanie-anderson wants to merge 7 commits into
masterfrom
chore/sdk-guidelines
Closed

stephanie-anderson wants to merge 7 commits into
masterfrom
chore/sdk-guidelines

Conversation

@stephanie-anderson

Copy link
Copy Markdown
Contributor

@vercel

vercel Bot commented May 11, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment May 11, 2026 2:59pm
sentry-docs Ready Ready Preview, Comment May 11, 2026 2:59pm

Request Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6ca2912. Configure here.


<SpecSection id="branch-naming" status="stable" since="1.0.0">

## Branch naming

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Removed section leaves broken links in referencing document

Medium Severity

The #branch-naming SpecSection was removed from code-submission.mdx, but opening-a-pr.mdx still links to it in two places (step 1 and the "Referenced Standards" list). These are now broken anchor links. Additionally, that playbook's step 1 still instructs contributors to follow the removed branch naming convention, making the guidance itself stale and contradictory.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 6ca2912. Configure here.

@coolguyzone coolguyzone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Changes look good to me, please just address the Cursor comment about the broken anchor link.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity.
It will be closed in 7 days if no further activity occurs.

If this PR is still relevant, please:

  • Push new commits, or
  • Leave a comment to keep it open

Thank you for your contribution!

@github-actions github-actions Bot added the Stale label Jun 28, 2026
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically closed due to inactivity.

If you'd like to continue working on this, feel free to reopen the PR or create a new one.

@github-actions github-actions Bot closed this Jul 6, 2026

This branch was successfully deployed

2 active deployments
Preview – sentry-docs — 4d2c6b32 Deployed May 11, 2026 by vercel[bot]
Preview – develop-docs — 4d2c6b32 Deployed May 11, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sdk-develop-docs PRs touching develop-docs/sdk Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants