Skip to content

Conversation

spikecurtis
Copy link
Contributor

@spikecurtis spikecurtis commented Sep 4, 2025

Adds guidelines about PR size to our contributing guide.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@spikecurtis spikecurtis added the docs Area: coder.com/docs label Sep 4, 2025 — with Graphite App
@spikecurtis spikecurtis marked this pull request as ready for review September 4, 2025 08:00
Copy link
Contributor

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

LGTM with nits

@@ -148,6 +148,32 @@ channel.

- [Frontend styling guide](./frontend.md#styling)

## Pull Requests
Copy link
Contributor

Choose a reason for hiding this comment

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

This section feels like it could absorb https://coder.com/docs/about/contributing/CONTRIBUTING#reviews, and make it more concise.

@spikecurtis spikecurtis force-pushed the spike/contributing-pr-size branch from b8f917a to 86d0b48 Compare September 8, 2025 05:48
@spikecurtis spikecurtis merged commit d25ff6c into main Sep 8, 2025
32 checks passed
Copy link
Contributor Author

Merge activity

@spikecurtis spikecurtis deleted the spike/contributing-pr-size branch September 8, 2025 05:55
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Area: coder.com/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants