Skip to content

Commit f92c38a

Browse files
pmcollinsemdnetoxrmx
authored
Add PR size guidelines to CONTRIBUTING.md (open-telemetry#4888)
* Add PR size guidelines to CONTRIBUTING.md * Address PR feedback * Apply suggestions from code review Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com> * Tweak size language * Tweak language --------- Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com> Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
1 parent 3fe2490 commit f92c38a

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

CONTRIBUTING.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,18 @@ file in the package it is testing. Make sure that the file name begins with
171171

172172
## Pull Requests
173173

174+
### How to Structure Pull Requests
175+
176+
Smaller PRs get merged faster, improve review quality, and reduce the risk of conflicts. Please keep PRs small and focused:
177+
178+
- **Aim for fewer than 500 lines** (excluding deletions, generated files, and test fixtures)
179+
- One change per PR -- address a single bug fix, feature, or refactor
180+
- If your work requires more extensive changes, consider a series of small, self-contained PRs instead.
181+
- If you want the community to understand how your small PR fits into the context of a larger change, consider putting
182+
the large change in your personal fork and providing a link to it in your PR description.
183+
- Strive to provide clear, incremental value to the repository in your PR, such that the repository is always in a
184+
functional, coherent state.
185+
174186
### How to Send Pull Requests
175187

176188
Everyone is welcome to contribute code to `opentelemetry-python` via GitHub

0 commit comments

Comments
 (0)