Thank you for your interest in deckhandler.
If you find an issue that interests you, please post a comment asking about it first. If you don't see that anyone has inquired about it, you can simply leave a comment that you are going to work on it. If people work on an issue without saying they are working on it, the result is sometimes three people submit a PR for the same issue.
If you find a problem for which no ticket has yet been created, please don't hesitate to open a new ticket, and let us know if you are going to work on that issue.
Please leave another note if you change your mind or if you get busy with other things and are unable to finish it. That lets us and other people know the ticket is available to be worked on by other people.
The goal is GNU style formatting
If you have no experience making pull requests and are confused about the process, you can practice at the repository Practice Git.
Note: That repo is not part of this project
Fork the repo (if you haven't already done so)
Clone it to your computer
When you're ready to work on an issue, be sure you're on the master branch. From there, create a separate branch (e.g. issue_32)
Make your changes. If you're unsure of some details while you're making edits, you can discuss them on the ticket.
If you are a first time contributor, please add your name and link of your choice to AUTHORS.
Commit your changes. git-cola is a nice GUI front-end for adding files and entering commit messages (git-cola is probably available from your OS repository).
Push the working branch (e.g. issue_32) to your remote fork and make your pull request.
- Do not merge it with the master branch on your fork. That would result in multiple, or unrelated patches being included in a single PR.
If any further changes need to be made, comments will be made on the pull request.
It's possible to work on two or more different patches (and therefore multiple branches) at one time, but it's recommended that beginners only work on one patch at a time.
Periodically, especially before starting a new patch, you'll need to sync your repo with the remote upstream. GitHub has instructions for doing this:
- For step 3 on that page, use https://github.com/theimpossibleastronaut/deckhandler for the URL.
- On that page, it shows how to merge the master branch (steps 4 & 5 of Syncing a Fork).