Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,4 @@

- this PR targets `main` from a non-`main` feature branch
- required CI is expected to pass before merge
- this PR is expected to land with squash merge
3 changes: 3 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,10 @@
4. Run the strongest local verification you can before opening a PR.
5. Open a pull request into `main`.
6. Merge only after required CI is green.
7. Merge with squash merge so each PR lands as one commit on `main`.

Direct pushes to `main` are not part of this repo workflow. The `main` branch is protected and changes should land through pull requests, including maintainers' own changes.
`main` also enforces linear history, so merge commits and rebase merges are not part of the repo workflow.

Suggested branch naming:

Expand Down Expand Up @@ -45,6 +47,7 @@ Use `make build-ios` when you need an iOS build check and your machine has the r
- include screenshots for UI changes when relevant
- keep the PR focused enough to review cleanly
- do not open a PR until the branch is in a state that can pass required CI
- expect the PR to land with squash merge, not merge commit or rebase merge

## Before Opening A PR

Expand Down
Loading