Skip to content

Commit cb119c3

Browse files
docs(sdk): Fix anchor references to review-ci
Update playbook links to match new section IDs after spec rewrite. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 010d267 commit cb119c3

8 files changed

Lines changed: 19 additions & 19 deletions

File tree

develop-docs/sdk/getting-started/playbooks/development/handling-a-regression.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ This helps identify gaps in testing, review processes, or monitoring that can be
7272

7373
- [Post-release monitoring](/sdk/getting-started/standards/coordination-maintenance#post-release-monitoring) — monitoring that detects regressions
7474
- [Rollback procedures](/sdk/getting-started/standards/coordination-maintenance#rollback-procedures) — emergency rollback process
75-
- [Release Gating Criteria](/sdk/getting-started/standards/review-ci#release-gating-criteria) — gates that may be expedited for critical patches
75+
- [Release Gating Criteria](/sdk/getting-started/standards/review-ci#release-gating) — gates that may be expedited for critical patches
7676

7777
---
7878

develop-docs/sdk/getting-started/playbooks/development/handling-external-contributor-pr.mdx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ Related resources:
3535

3636
#### 1. Triage within 2 business days
3737

38-
([Review SLAs](/sdk/getting-started/standards/review-ci#review-slas)). Every external PR **REQUIRES** a timely, substantive response.
38+
([Review SLAs](/sdk/getting-started/standards/review-ci#code-review)). Every external PR **REQUIRES** a timely, substantive response.
3939

4040
#### 2. Check for a linked issue
4141

@@ -55,16 +55,16 @@ Review it using the standard process ([Reviewing a PR](/sdk/getting-started/play
5555

5656
#### 6. When requesting changes
5757

58-
You **MUST** be specific, use LOGAF prefixes ([Review feedback conventions](/sdk/getting-started/standards/review-ci#review-feedback-conventions)), and explain *why* — so the contributor (or their AI tool) can address the feedback effectively.
58+
You **MUST** be specific, use LOGAF prefixes ([Review feedback conventions](/sdk/getting-started/standards/review-ci#code-review)), and explain *why* — so the contributor (or their AI tool) can address the feedback effectively.
5959

6060
#### 7. Closing fast is kind
6161

6262
A 30-second close with a clear, respectful reason is better than a PR that sits open for weeks with no response. That respects everyone's time.
6363

6464
## Referenced Standards
6565

66-
- [Review SLAs](/sdk/getting-started/standards/review-ci#review-slas) — timely response requirements
67-
- [Review feedback conventions](/sdk/getting-started/standards/review-ci#review-feedback-conventions) — LOGAF scale for feedback
66+
- [Review SLAs](/sdk/getting-started/standards/review-ci#code-review) — timely response requirements
67+
- [Review feedback conventions](/sdk/getting-started/standards/review-ci#code-review) — LOGAF scale for feedback
6868
- [PR description quality](/sdk/getting-started/standards/code-submission#pr-description-quality) — linked issue requirement
6969

7070
---

develop-docs/sdk/getting-started/playbooks/development/opening-a-pr.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ When green, mark the PR as ready for review.
7575

7676
#### 9. Assign 1–2 reviewers
7777

78-
If the PR touches public API, dependencies, or security-sensitive areas, you **MUST** assign an @sdk-seniors reviewer ([Required reviewers](/sdk/getting-started/standards/review-ci#required-reviewers)).
78+
If the PR touches public API, dependencies, or security-sensitive areas, you **MUST** assign an @sdk-seniors reviewer ([Required reviewers](/sdk/getting-started/standards/review-ci#code-review)).
7979

8080
## Referenced Standards
8181

@@ -87,7 +87,7 @@ If the PR touches public API, dependencies, or security-sensitive areas, you **M
8787
- [Changelog entry](/sdk/getting-started/standards/code-submission#changelog-entry) — user-facing change documentation
8888
- [One logical change per PR](/sdk/getting-started/standards/code-submission#one-logical-change-per-pr) — focused PR scope
8989
- [Documentation-with-code](/sdk/getting-started/standards/repository-docs#documentation-with-code) — docs PR requirements
90-
- [Required reviewers](/sdk/getting-started/standards/review-ci#required-reviewers)@sdk-seniors review triggers
90+
- [Required reviewers](/sdk/getting-started/standards/review-ci#code-review)@sdk-seniors review triggers
9191
- [Test requirements by change type](/sdk/getting-started/standards/code-quality#test-requirements-by-change-type) — test coverage requirements
9292

9393
---

develop-docs/sdk/getting-started/playbooks/development/reviewing-a-pr.mdx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,11 +63,11 @@ You **SHOULD** use the [`sentry-skills:code-review`](https://github.com/getsentr
6363

6464
#### 4. Check for @sdk-seniors review triggers
6565

66-
([Required reviewers](/sdk/getting-started/standards/review-ci#required-reviewers)): public API changes, new dependencies, schema changes, security-sensitive code, new frameworks. If any apply and no @sdk-seniors reviewer is assigned, flag it.
66+
([Required reviewers](/sdk/getting-started/standards/review-ci#code-review)): public API changes, new dependencies, schema changes, security-sensitive code, new frameworks. If any apply and no @sdk-seniors reviewer is assigned, flag it.
6767

6868
#### 5. Use LOGAF prefixes on all feedback
6969

70-
You **MUST** use LOGAF prefixes on all feedback ([Review feedback conventions](/sdk/getting-started/standards/review-ci#review-feedback-conventions)):
70+
You **MUST** use LOGAF prefixes on all feedback ([Review feedback conventions](/sdk/getting-started/standards/review-ci#code-review)):
7171
- `h:` (high) — must fix before merge. Bugs, security issues, breakage, data loss.
7272
- `m:` (medium) — should fix. Design concerns, missing tests, unclear code.
7373
- `l:` (low) — optional nit. Style preferences, minor suggestions.
@@ -78,9 +78,9 @@ You **MUST NOT** block for style preferences. The goal is risk reduction, not pe
7878

7979
## Referenced Standards
8080

81-
- [Review feedback conventions](/sdk/getting-started/standards/review-ci#review-feedback-conventions) — LOGAF scale and blocking criteria
82-
- [Required reviewers](/sdk/getting-started/standards/review-ci#required-reviewers)@sdk-seniors review triggers
83-
- [Required CI checks baseline](/sdk/getting-started/standards/review-ci#required-ci-checks-baseline) — minimum CI requirements
81+
- [Review feedback conventions](/sdk/getting-started/standards/review-ci#code-review) — LOGAF scale and blocking criteria
82+
- [Required reviewers](/sdk/getting-started/standards/review-ci#code-review)@sdk-seniors review triggers
83+
- [Required CI checks baseline](/sdk/getting-started/standards/review-ci#required-ci-checks) — minimum CI requirements
8484
- [Test requirements by change type](/sdk/getting-started/standards/code-quality#test-requirements-by-change-type) — test coverage expectations
8585
- [Test quality](/sdk/getting-started/standards/code-quality#test-quality) — meaningful assertion requirements
8686
- [PR description quality](/sdk/getting-started/standards/code-submission#pr-description-quality) — description content requirements

develop-docs/sdk/getting-started/playbooks/development/reviewing-ai-generated-code.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ Public API, dependencies, schema changes, security-sensitive code, frameworks.
5353

5454
#### 5. Use LOGAF prefixes on feedback
5555

56-
([Review feedback conventions](/sdk/getting-started/standards/review-ci#review-feedback-conventions))
56+
([Review feedback conventions](/sdk/getting-started/standards/review-ci#code-review))
5757

5858
#### 6. Approve when only `l:` items remain
5959

@@ -91,7 +91,7 @@ You **SHOULD** use the [`sentry-skills:find-bugs`](https://github.com/getsentry/
9191

9292
## Referenced Standards
9393

94-
- [Review feedback conventions](/sdk/getting-started/standards/review-ci#review-feedback-conventions) — LOGAF scale and blocking criteria
94+
- [Review feedback conventions](/sdk/getting-started/standards/review-ci#code-review) — LOGAF scale and blocking criteria
9595
- [Test requirements by change type](/sdk/getting-started/standards/code-quality#test-requirements-by-change-type) — test coverage expectations
9696
- [Test quality](/sdk/getting-started/standards/code-quality#test-quality) — meaningful assertion requirements
9797
- [AI attribution](/sdk/getting-started/standards/code-submission#ai-attribution) — Co-Authored-By footer requirement

develop-docs/sdk/getting-started/playbooks/sdk-lifecycle/cutting-a-release.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ Related resources:
3939

4040
#### 1. Verify release gating criteria
4141

42-
You **MUST** verify all criteria pass ([Release gating criteria](/sdk/getting-started/standards/review-ci#release-gating-criteria)):
42+
You **MUST** verify all criteria pass ([Release gating criteria](/sdk/getting-started/standards/review-ci#release-gating)):
4343
- All required CI checks pass on the default branch
4444
- No unresolved `h:` (high) review comments on merged PRs since last release
4545
- Changelog is substantive (not empty or internal-only for a release with user-facing changes)
@@ -111,7 +111,7 @@ Common SDK-specific variations include:
111111

112112
## Referenced Standards
113113

114-
- [Release gating criteria](/sdk/getting-started/standards/review-ci#release-gating-criteria) — pre-release quality gates
114+
- [Release gating criteria](/sdk/getting-started/standards/review-ci#release-gating) — pre-release quality gates
115115
- [Post-release monitoring](/sdk/getting-started/standards/coordination-maintenance#post-release-monitoring) — monitoring requirements and timeline
116116
- [Rollback procedures](/sdk/getting-started/standards/coordination-maintenance#rollback-procedures) — emergency rollback process
117117

develop-docs/sdk/getting-started/playbooks/setup/setting-up-new-sdk-repo.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ Related resources:
4141

4242
Create the repository in the `getsentry` organization. You **MUST** configure branch protection rules:
4343
- Require CI checks to pass ([Required CI checks baseline](/sdk/getting-started/standards/review-ci#required-ci-checks))
44-
- Require at least one approving review ([Required reviewers](/sdk/getting-started/standards/review-ci#required-reviewers))
44+
- Require at least one approving review ([Required reviewers](/sdk/getting-started/standards/review-ci#code-review))
4545

4646
You **MUST** set up a `CODEOWNERS` file with team members to automatically request reviews.
4747

@@ -109,7 +109,7 @@ If the tool cannot do this, the documentation needs work.
109109
- [CONTRIBUTING.md](/sdk/getting-started/standards/repository-docs#contributing-md) — contributor guidelines
110110
- [PR template](/sdk/getting-started/standards/repository-docs#pr-template) — pull request template structure
111111
- [AI context file maintenance](/sdk/getting-started/standards/repository-docs#ai-context-maintenance) — maintaining AI-readable documentation
112-
- [Required reviewers](/sdk/getting-started/standards/review-ci#required-reviewers) — review approval requirements
112+
- [Required reviewers](/sdk/getting-started/standards/review-ci#code-review) — review approval requirements
113113
- [Required CI checks baseline](/sdk/getting-started/standards/review-ci#required-ci-checks) — minimum CI pipeline requirements
114114
- [Commit message format](/sdk/getting-started/standards/code-submission#commit-message-format) — conventional commit format
115115
- [Linting and formatting in CI](/sdk/getting-started/standards/code-quality#linting-formatting-ci) — automated code style checks

develop-docs/sdk/getting-started/playbooks/setup/setting-up-release-infrastructure.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ For the ongoing release process after setup, see "Cutting a release" (wip).
129129

130130
- [Version format](/sdk/getting-started/standards/release-versioning#version-format) — SemVer requirements
131131
- [Release tooling](/sdk/getting-started/standards/release-versioning#release-tooling) — Craft and publish setup
132-
- [Release gating criteria](/sdk/getting-started/standards/review-ci#release-gating-criteria) — Pre-release validation requirements
132+
- [Release gating criteria](/sdk/getting-started/standards/review-ci#release-gating) — Pre-release validation requirements
133133
- [Rollback procedures](/sdk/getting-started/standards/coordination-maintenance#rollback-procedures) — Emergency release rollback process
134134

135135
---

0 commit comments

Comments
 (0)