[pulsar-client-cpp] branch main updated: Add pull request template (#8)

[mm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mmerli pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/pulsar-client-cpp.git


The following commit(s) were added to refs/heads/main by this push:
     new 6cd205b  Add pull request template (#8)
6cd205b is described below

commit 6cd205b11331f271883c9670c5a47ebc76d419ff
Author: Kai Wang <kw...@streamnative.io>
AuthorDate: Thu Sep 29 20:52:26 2022 +0800

    Add pull request template (#8)
---
 .github/PULL_REQUEST_TEMPLATE.md | 65 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 65 insertions(+)

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
new file mode 100644
index 0000000..4bcbe00
--- /dev/null
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -0,0 +1,65 @@
+<!--
+### Contribution Checklist
+  
+  - PR title format should be *[type][component] summary*. For details, see *[Guideline - Pulsar PR Naming Convention](https://docs.google.com/document/d/1d8Pw6ZbWk-_pCKdOmdvx9rnhPiyuxwq60_TrD68d7BA/edit#heading=h.trs9rsex3xom)*. 
+
+  - Fill out the template below to describe the changes contributed by the pull request. That will give reviewers the context they need to do the review.
+  
+  - Each pull request should address only one issue, not mix up code from multiple issues.
+  
+  - Each commit in the pull request has a meaningful commit message
+
+  - Once all items of the checklist are addressed, remove the above text and this checklist, leaving only the filled out template below.
+-->
+
+<!-- Either this PR fixes an issue, -->
+
+Fixes #<xyz>
+
+<!-- or this PR is one task of an issue -->
+
+Master Issue: #<xyz>
+
+### Motivation
+
+<!-- Explain here the context, and why you're making that change. What is the problem you're trying to solve. -->
+
+### Modifications
+
+<!-- Describe the modifications you've done. -->
+
+### Verifying this change
+
+- [ ] Make sure that the change passes the CI checks.
+
+*(Please pick either of the following options)*
+
+This change is a trivial rework / code cleanup without any test coverage.
+
+*(or)*
+
+This change is already covered by existing tests, such as *(please describe tests)*.
+
+*(or)*
+
+This change added tests and can be verified as follows:
+
+*(example:)*
+  - *Added integration tests for end-to-end deployment with large payloads (10MB)*
+  - *Extended integration test for recovery after broker failure*
+
+### Documentation
+
+<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
+
+- [ ] `doc-required` 
+(Your PR needs to update docs and you will update later)
+
+- [ ] `doc-not-needed` 
+(Please explain why)
+
+- [ ] `doc` 
+(Your PR contains doc changes)
+
+- [ ] `doc-complete`
+(Docs have been already added)