About Features Downloads Getting Started Documentation Events Support GitHub

Love VuFind®? Consider becoming a financial supporter. Your support helps build a better VuFind®!

Site Tools


Warning: This page has not been updated in over over a year and may be outdated or deprecated.
development:making_pull_requests

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
development:making_pull_requests [2021/04/13 11:35] – [Making Pull Requests] demiankatzdevelopment:making_pull_requests [2022/04/11 16:16] (current) – [What Happens to a Pull Request] emaijala
Line 58: Line 58:
   * Accessibility   * Accessibility
      
-You may receive a request for further information or clarification, suggestions for refinement and other constructive comments that aim to maintain and improve quality. Humans, nonetheless, are fallible, so feel free to question the comments if you don't find them reasonable.+You may receive a request for further information or clarification, suggestions for refinement and other constructive comments that aim to maintain and improve quality. Humans, nonetheless, are fallible, so feel free to question the comments if you don't find them reasonable.  
 + 
 +If you've done changes requested in a review, please re-request review from the same person. This ensures that the pull request gets attention.
  
 When everything checks out and all requirements are fulfilled, the pull request will get merged. After it has been merged, you can delete your own branch. It's a good idea to clean up obsolete branches regularly. When everything checks out and all requirements are fulfilled, the pull request will get merged. After it has been merged, you can delete your own branch. It's a good idea to clean up obsolete branches regularly.
Line 101: Line 103:
  
 ---- struct data ---- ---- struct data ----
 +properties.Page Owner : 
 ---- ----
  
development/making_pull_requests.1618313737.txt.gz · Last modified: 2021/04/13 11:35 by demiankatz