Thank you for sharing your code with the Tessera project. We appreciate your contribution!
If you're not already on the Tessera developers list, take a minute to join. This is as easy as sending an email to [email protected]. It would be great if you'd introduce yourself to the group but it's not required. You can just let your code do the talking for you if you like.
Before you write too much code, check the open issues in the install-emr issue tracker to see if someone else has already filed an issue related to your work or is already working on it. If not, go ahead and open a new issue.
Shoot us a quick email on the mailing list letting us know what you're working on. There will likely be people on the list who can give you tips about where to find relevant source or alert you to other planned changes that might effect your work.
Github provides a nice overview on how to create a pull request.
Some general rules to follow:
- Do your work in a fork of the install-emr repo.
- Create a branch for each feature/bug in install-emr that you're working on. These branches are often called "feature" or "topic" branches.
- Use your feature branch in the pull request. Any changes that you push to your feature branch will automatically be shown in the pull request. If your feature branch is not based off the latest master, you will be asked to rebase it before it is merged.
- If your pull request fixes an issue, reference the issue so that it will be closed when your pull request is merged
- Keep your pull requests as small as possible. Large pull requests are hard to review. Try to break up your changes into self-contained and incremental pull requests, if need be, and reference dependent pull requests, e.g. "This pull request builds on request #92. Please review #92 first."
- The first line of commit messages should be a short (<80 character) summary, followed by an empty line and then, optionally, any details that you want to share about the commit.