alamb opened a new pull request, #7749:
URL: https://github.com/apache/arrow-rs/pull/7749

   # Which issue does this PR close?
   
   N/A
   
   # Rationale for this change
   
   It is critical and generally required to add tests for any changes to 
arrow-rs and it something we look for in our PR reviews. It would be nice to 
remind people of this explicitly in the PR 
   
   # What changes are included in this PR?
   
   1. Update the PR template to include a section on testing
   2. Add a list marker (`-`) to the closes section which causes github to 
render the name of the PR in markdown not just the number
   
   I copied the wording from DataFusion: 
https://github.com/apache/datafusion/blob/b6c8cc57760686fffe4878e69c1be27e4d9f5e68/.github/pull_request_template.md?plain=1#L22
   
   
   # Are there any user-facing changes?
   
   A new section on PR descriptions


-- 
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.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to