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.
Both sides previous revisionPrevious revisionNext revision | Previous revisionNext revisionBoth sides next revision | ||
development:making_pull_requests [2021/04/13 06:45] – Add suggestions and more information emaijala | development:making_pull_requests [2021/04/13 11:31] – [Steps to Making the Pull Request] demiankatz | ||
---|---|---|---|
Line 5: | Line 5: | ||
Anyone may and is encouraged to make and contribute to pull requests. Since VuFind is so flexible, a lot of libraries add custom functionality to their instances. We're thrilled when developers share this code back with us! | Anyone may and is encouraged to make and contribute to pull requests. Since VuFind is so flexible, a lot of libraries add custom functionality to their instances. We're thrilled when developers share this code back with us! | ||
- | On this page we provide a few suggestions | + | On this page we provide a few suggestions |
===== Contents of a Good Pull Request ===== | ===== Contents of a Good Pull Request ===== | ||
Line 33: | Line 33: | ||
- First time only: " | - First time only: " | ||
- | - Create a new branch based on the target branch (normally dev) and give it a desciptive | + | - Create a new branch based on the target branch (normally dev) and give it a descriptive |
- Make the changes in this new branch, commit and push them to your repository in GitHub. | - Make the changes in this new branch, commit and push them to your repository in GitHub. | ||
- | - Go to GitHub and start a new pull request. | + | - Go to GitHub and start a new pull request |
- Make sure base repository and base are correct. | - Make sure base repository and base are correct. | ||
- Enter a description about what has been done and why. It's often useful to provide a short use-case when it's not obvious from the change. | - Enter a description about what has been done and why. It's often useful to provide a short use-case when it's not obvious from the change. | ||
Line 41: | Line 41: | ||
- Submit the pull request. | - Submit the pull request. | ||
| | ||
- | You may also want to [[development: | + | You may also want to [[development: |
==== What Happens to a Pull Request ==== | ==== What Happens to a Pull Request ==== |
development/making_pull_requests.txt · Last modified: 2022/04/11 16:16 by emaijala