You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ag...@apache.org on 2022/11/07 23:31:51 UTC

[arrow-datafusion] branch master updated: Template change (#4114)

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

agrove pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/master by this push:
     new 531dfdc21 Template change (#4114)
531dfdc21 is described below

commit 531dfdc21178b3f0a2ca1b188c56e26143d7af87
Author: Brent Gardner <br...@spaceandtime.io>
AuthorDate: Mon Nov 7 16:31:47 2022 -0700

    Template change (#4114)
    
    * Template change
    
    * Update .github/pull_request_template.md
    
    Co-authored-by: Andrew Lamb <an...@nerdnetworks.org>
    
    Co-authored-by: Andrew Lamb <an...@nerdnetworks.org>
---
 .github/pull_request_template.md | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
index 0b9ff3a46..ec66a1270 100644
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -19,6 +19,16 @@ Closes #.
 There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR.
 -->
 
+# Are these changes tested?
+
+<!--
+We typically require tests for all PRs in order to:
+1. Prevent the code from being accidentally broken by subsequent changes
+2. Serve as another way to document the expected behavior of the code
+
+If tests are not included in your PR, please explain why (for example, are they covered by existing tests)?
+-->
+
 # Are there any user-facing changes?
 
 <!--