-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Pull Requests
Rules for reviewing, creating, or modifying a pull request.
See also: Hacking Guide
Open PRs will be landed if approved. If the PR is not yet ready to land (e.g. blocked by another PR or WIP) then use draft mode to signal to others that the PR is not yet ready to land.
If another PR needs to be landed first, use the term "Blocked by" at the top of the PR to indicate that another PR blocks the current PR and needs to be landed first. Do not rely on this alone to block the land of a PR as it is easily missed; you should also set it to draft.
Rebasing keeps the commit history clean and linear, making it easier to track changes and avoiding unnecessary merge commits that clutter the history.
Using git commit --amend allows you to update the original commit with the corrected code, keeping the commit history cleaner and more concise. This avoids having multiple commits like "fix typo" or "fix previous commit."
Squashing allows casual commits in a PR, which we do not want.
If the PR description is LLM-generated, then copy/paste:
Your PR description reads like a predictive text matrix that lost its context window.
We require manually written, carbon-based PR descriptions, not generated token mush.
Please read the following resources:
- [Our hacking guide](https://github.com/deskflow/deskflow/wiki/Hacking-Guide)
- [RFC 406i (RAGS)](https://406.fail)
For when the PR code diff is LLM-generated, then copy/paste:
Your PR code diff reads like a predictive text matrix that lost its context window.
We require manual, carbon-based testing and actual logical continuity, not automated guessing games.
Please read the following resources:
- [Our hacking guide](https://github.com/deskflow/deskflow/wiki/Hacking-Guide)
- [RFC 406i (RAGS)](https://406.fail)