👍🎉 First off, thanks for taking the time to contribute! 🎉👍
The following is a set of guidelines (not rules) for contributing to Atomic App, which is hosted in the Project Atomic Organization on GitHub. These are just guidelines, not rules, use your best judgment and feel free to propose changes to this document in a pull request.
- You can create an issue here, include as many details as possible with your report.
- Include the version of Atomic App you are using, have a look at the docker container image tag.
- Include the behavior you expected and maybe other places you've seen that behavior
- Perform a cursory search to see if a similar issue has already been submitted
Before you submit your pull request consider the following guidelines:
-
Make your changes in a new git branch:
git checkout -b bug/my-fix-branch master
-
Create your patch, including appropriate test cases. Do not forget to add a copyright notice to your files, pls read along the line 625 of gpl-3.txt
-
Commit your changes using a descriptive commit message.
-
Push your branch to GitHub:
git push origin bug/my-fix-branch
-
In GitHub, send a pull request to
atomicapp:master
. -
If we suggest changes then:
-
Make the required updates.
-
Rebase your branch and force push to your GitHub repository (this will update your Pull Request):
git rebase master -i git push origin bug/my-fix-branch -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 upstream repository:
-
Delete the remote branch on GitHub either through the GitHub web UI or your local shell as follows:
git push origin --delete bug/my-fix-branch
-
Check out the master branch:
git checkout master -f
-
Delete the local branch:
git branch -D bug/my-fix-branch
-
Update your master with the latest upstream version:
git pull --ff upstream master
- Use the present tense ("Add feature" not "Added feature")
- Use the imperative mood ("Move cursor to..." not "Moves cursor to..."):wq
- Reference issues and pull requests liberally