Skip to content

Commit bc3fdad

Browse files
chore: ask in the PR template whether a change is breaking (#3782)
The template now asks every PR whether it breaks existing apps, and if it does, asks for both markers. It mentions the commit subject as well as the PR title because a one-commit PR lands on main with its commit's subject.
1 parent 110dac1 commit bc3fdad

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

‎PULL_REQUEST_TEMPLATE.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ before sending PRs. We cannot accept code without this.
2525
- Issue number for this PR: #nnn (required)
2626
- Docs included?: (yes/no; required for all API/functional changes)
2727
- Test units included?: (yes/no; required)
28+
- Breaking change?: (yes/no, required. If yes, add a `BREAKING CHANGE:` footer to your commit message that says what an upgrading app must change, and put `!` before the colon in the PR title and the commit subject, like `fix(auth)!: ...`)
2829
- In a clean directory, `yarn install`, `yarn test` run successfully? (yes/no; required)
2930

3031
### Description

0 commit comments

Comments
 (0)