@@ -6,11 +6,12 @@ This guide documents the PR description style used in the Coder repository, base
66
77Follow [ Conventional Commits 1.0.0] ( https://www.conventionalcommits.org/en/v1.0.0/ ) format:
88
9- ```
9+ ``` text
1010type(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
4245Previously, when a devcontainer config file was modified, the dirty
4346status 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
5155Changes from https://github.com/coder/aibridge/pull/71/
5256```
5357
5458** Example from PR #21077 :**
59+
5560``` markdown
5661Removes references to adding database replicas from the scaling docs,
5762as 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
7884Change `@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
1191262 . ** 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
1251333 . ** 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
1311404 . ** 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+
201212See PR #21099 - includes query timing metrics and explains the index solution
202213
203214### Bug Fixes
215+
204216See PR #21100 - describes broken behavior then the fix in two sentences
205217
206218### Documentation
219+
207220See PR #21080 - long form explaining inaccuracies and improvements for major rewrite
208221See PR #21077 - one sentence for simple correction
209222
210223### Features
224+
211225See PR #21106 - simple statement of what was added and dependencies
212226
213227### Refactoring
228+
214229See PR #21102 - explains why client-side sorting is now redundant
215230
216231### Configuration
232+
217233See PR #21083 - adds guidelines with issue reference
0 commit comments