diff options
author | James E. King III <jking@apache.org> | 2019-02-14 17:58:04 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-02-14 17:58:04 -0500 |
commit | b000bc02df16a16a21d0fe3467227c3bc6d62a0c (patch) | |
tree | ec88ab121a1957bcdb20ecfd2af0f775afde98ef /.github | |
parent | dbc1f8def5018ce5d85d38b9875c6c6b6b424478 (diff) | |
download | thrift-b000bc02df16a16a21d0fe3467227c3bc6d62a0c.tar.gz |
Update the pull request template [ci skip]
Diffstat (limited to '.github')
-rw-r--r-- | .github/pull_request_template.md | 37 |
1 files changed, 3 insertions, 34 deletions
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index a6b4382b4..edd7befe8 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,43 +1,12 @@ <!-- Explain the changes in the pull request below: --> - <!-- We recommend you review the checklist before submitting a pull request. --> +- [ ] Did you create an [Apache Jira](https://issues.apache.org/jira/projects/THRIFT/issues/) ticket? (not required for trivial changes) +- [ ] Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"? (not required for trivial changes) - [ ] Did you squash your changes to a single commit? - -- [ ] Do you need an [Apache Jira](https://issues.apache.org/jira/projects/THRIFT/issues/) ticket?<details><summary>Expand for guidance...</summary> - - `Yes` if your change requires a release note. - - `Yes` if your change is a breaking change. - - `No` if you change is trivial, such as fixing a typo. -</details> - -- [ ] Is this change worthy of a release note? <details><summary>Examples of Release Note-worthy examples...</summary> - - Breaking Changes - - New, Deprecated, or Removed Languages - - Security Fixes - - Significant Refactoring - - Changing how the product is built -</details> - -- [ ] Breaking changes have additional requirements: <details><summary>Expand for instructions...</summary> - - Add or reference an existing Apache Jira THRIFT ticket. - - Add a `Breaking-Change` label to the Jira ticket. - - Add a note to the `lib/<language>/README.md` file. - - Add a line to the `CHANGES.md` file. -</details> - -- [ ] Does this change require a build? <details><summary>Expand for guidance...</summary> - - `Yes` for any code change - - `Yes` for any build script change - - `Yes` for any docker build environment change - - `Yes` for any change affecting the cross test suite - - `No` for documentation-only changes - - `No` for trivial changes, for example fixing a typo. - <br/> - If your change does not require a build, you can add [ci skip] to the end of your commit message.<br/> - This will avoid costly and unnecessary builds in both the pull request and once it is merged. -</details> +- [ ] Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change" <!-- The Contributing Guide at: |