Skip to content

Commit 2bc9bec

Browse files
committed
fix: add markdown linting fixes to PR style guide
1 parent 34c8170 commit 2bc9bec

File tree

1 file changed

+17
-1
lines changed

1 file changed

+17
-1
lines changed

.claude/docs/PR_STYLE_GUIDE.md

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,12 @@ This guide documents the PR description style used in the Coder repository, base
66

77
Follow [Conventional Commits 1.0.0](https://www.conventionalcommits.org/en/v1.0.0/) format:
88

9-
```
9+
```text
1010
type(scope): brief description
1111
```
1212

1313
**Common types:**
14+
1415
- `feat`: New features
1516
- `fix`: Bug fixes
1617
- `refactor`: Code refactoring without behavior change
@@ -19,6 +20,7 @@ type(scope): brief description
1920
- `chore`: Dependency updates, tooling changes
2021

2122
**Examples:**
23+
2224
- `feat: add tracing to aibridge`
2325
- `fix: move contexts to appropriate locations`
2426
- `perf(coderd/database): add index on workspace_app_statuses.app_id`
@@ -38,6 +40,7 @@ Most PRs use a simple 1-2 paragraph format:
3840
```
3941

4042
**Example from PR #21100:**
43+
4144
```markdown
4245
Previously, when a devcontainer config file was modified, the dirty
4346
status was updated internally but not broadcast to websocket listeners.
@@ -47,11 +50,13 @@ websocket listeners immediately when a config file changes.
4750
```
4851

4952
**Example from PR #21085:**
53+
5054
```markdown
5155
Changes from https://github.com/coder/aibridge/pull/71/
5256
```
5357

5458
**Example from PR #21077:**
59+
5560
```markdown
5661
Removes references to adding database replicas from the scaling docs,
5762
as Coder only allows a single connection URL. These passages where added in error.
@@ -73,6 +78,7 @@ How the solution works
7378
```
7479

7580
**Example from PR #21101:**
81+
7682
```markdown
7783
## Summary
7884
Change `@Tags` from `Organizations` to `Enterprise` for POST /licenses...
@@ -101,6 +107,7 @@ Refs #[issue-number]
101107
```
102108

103109
**Example from PR #21080** (dev containers docs rewrite):
110+
104111
- Started with "This PR rewrites the dev containers documentation for GA readiness"
105112
- Listed specific inaccuracies being fixed
106113
- Explained organizational changes
@@ -117,18 +124,21 @@ Refs #[issue-number]
117124
- `Refs #XXX` (for general reference)
118125

119126
2. **Performance Context** (when relevant)
127+
120128
```markdown
121129
Each query took ~30ms on average with 80 requests/second to the cluster,
122130
resulting in ~5.2 query-seconds every second.
123131
```
124132

125133
3. **Migration Warnings** (when relevant)
134+
126135
```markdown
127136
**NOTE**: This migration creates an index on `workspace_app_statuses`.
128137
For deployments with heavy task usage, this may take a moment to complete.
129138
```
130139

131140
4. **Visual Evidence** (for UI changes)
141+
132142
```markdown
133143
<img width="1281" height="425" alt="image" src="..." />
134144
```
@@ -198,20 +208,26 @@ Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
198208
## Examples by Category
199209

200210
### Performance Improvements
211+
201212
See PR #21099 - includes query timing metrics and explains the index solution
202213

203214
### Bug Fixes
215+
204216
See PR #21100 - describes broken behavior then the fix in two sentences
205217

206218
### Documentation
219+
207220
See PR #21080 - long form explaining inaccuracies and improvements for major rewrite
208221
See PR #21077 - one sentence for simple correction
209222

210223
### Features
224+
211225
See PR #21106 - simple statement of what was added and dependencies
212226

213227
### Refactoring
228+
214229
See PR #21102 - explains why client-side sorting is now redundant
215230

216231
### Configuration
232+
217233
See PR #21083 - adds guidelines with issue reference

0 commit comments

Comments
 (0)