mirror of
https://github.com/fleetdm/fleet
synced 2026-05-23 17:08:53 +00:00
Amend community PR process. (#25608)
This commit is contained in:
parent
7be88041e1
commit
d5e4825f0f
1 changed files with 7 additions and 2 deletions
|
|
@ -222,12 +222,17 @@ If you're assigned a community pull request for review, it is important to keep
|
|||
|
||||
If the PR is a quick fix (i.e. typo) or obvious technical improvement that doesn't change the product, it can be merged.
|
||||
|
||||
For PRs that change the product:
|
||||
**For PRs that change the product:**
|
||||
|
||||
- Assign the PR to the appropriate product group EM (Engineering Manager).
|
||||
- Notify the EM in the #help-engineering Slack channel.
|
||||
|
||||
The EM will be the contact point for the contributor and will ensure the PR is reviewed by the appropriate team member when ready. The EM should:
|
||||
|
||||
- Set the PR to draft.
|
||||
- Thank the contributor for their hard work, explain that all changes to the product go through Fleet's [prioritization process](https://fleetdm.com/handbook/company/product-groups#how-feature-requests-are-prioritized), and ask them to file a [feature request](https://github.com/fleetdm/fleet/issues/new?assignees=&labels=%3Aproduct&projects=&template=feature-request.md&title=) that describe the change their PR is introducing.
|
||||
|
||||
For PRs that will not be merged:
|
||||
**For PRs that will not be merged:**
|
||||
|
||||
- Thank the contributor for their effort and explain why the changes won't be merged.
|
||||
- Close the PR.
|
||||
|
|
|
|||
Loading…
Reference in a new issue