BryanCutler commented on a change in pull request #25310: [SPARK-28578][INFRA] Improve Github pull request template URL: https://github.com/apache/spark/pull/25310#discussion_r311171074
########## File path: .github/PULL_REQUEST_TEMPLATE ########## @@ -1,10 +1,40 @@ -## What changes were proposed in this pull request? +<!-- +Thanks for sending a pull request! Here are some tips for you: + 1. If this is your first time, please read our contributor guidelines: https://spark.apache.org/contributing.html + 2. Ensure you have added or run the appropriate tests for your PR: https://spark.apache.org/developer-tools.html + 3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., '[WIP][SPARK-XXXX] Your PR title ...'. + 4. Update your PR description whenever it is needed. + 5. Provide a concise example to reproduce the issue if possible for a faster review. +--> -(Please fill in changes proposed in this fix) +### What changes were proposed in this pull request? +<!-- +Please clarify what changes you are proposing. The purpose of this section is to outline the changes and how this PR fixes the issue. +Therefore, if possible, please consider write some useful notes for better and faster reviews in your PR. See the examples below. Review comment: I would remove `Therefore`. Typo: `write` -> `writing`. Also remove `some` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org