Thank you for contributing! GoFortune welcomes contributions following the guidelines described in this document.
This document outlines the procedures and what to expect when contributing documentation, a bug report, a feature request, or new code via a pull request. I
Please read and follow these guidelines before submitting a bug report, feature request or pull request.
- Code of Conduct
- Question or Problem?
- Issues and Bugs
- Feature Requests
- Submission Guidelines
- Coding Rules
- Signing the CLA
Make sure you help to keep GoFortune open and inclusive. Please read and follow our Code of Conduct.
If you have questions or a problem related with GoFortune, please direct these to the issue tracker. Although it seems clear an issue tracker is not the best communication mechanism, for the time being there is no need for any other mechanism.
If you find a bug in the source code or a mistake in the documentation, you can help by submitting an issue to our GitHub Repository. Even better you can submit a Pull Request with a fix.
Please see the Submission Guidelines below.
You can request a new feature by submitting an issue to our GitHub Repository. If you would like to implement a new feature then consider a Pull Request.
Please see the Submission Guidelines below.
Before you submit your issue search the archive, maybe your question was already answered.
If your issue appears to be a bug, and hasn't been reported, open a new issue.
Before you submit your pull request consider the following guidelines:
-
Search GitHub for an open or closed Pull Request that relates to your submission. You don't want to duplicate effort.
-
Please sign our Contributor License Agreement before sending pull requests. Code cannot be accepted without this.
-
Make your changes in a new git branch:
git checkout -b feature/my-new-feature master
-
Create your patch, including appropriate test cases.
-
Follow our Coding Rules.
-
Run the full test suite and ensure that all tests pass.
-
Commit your changes using a descriptive commit message.
-
Push your branch to GitHub:
In GitHub, send a pull request to vromero/gofortune:master
.
If the PR gets too outdated it may be asked to rebase and force push to update the PR:
git rebase master -i
git push origin feature/my-new-feature -f
That's it! Thank you for your contribution!
After your pull request is merged, you can safely delete your branch and pull the changes from the main (upstream) repository:
-
Delete the remote branch on GitHub either through the GitHub web UI or your local shell as follows:
git push origin --delete feature/my-new-feature
-
Check out the master branch:
git checkout master -f
-
Delete the local branch:
git branch -D feature/my-new-feature
-
Update your master with the latest upstream version:
git pull --ff upstream master
To ensure consistency throughout the source code, keep these rules in mind as you are working:
- All features or bug fixes must be tested by one or more tests.
- All public API methods must be documented.
- With the exceptions listed below, we follow the rules contained in [Google's Go Style Guide][https://github.com/golang/go/wiki/CodeReviewComments]:
Please sign our Contributor License Agreement (CLA) before sending pull requests. For any code changes to be accepted, the CLA must be signed. It's a quick process, we promise!