|
| 1 | +# Contributors Guide |
| 2 | + |
| 3 | +Interested in contributing? Awesome! Before you do though, please read our |
| 4 | +[Code of Conduct](https://slackhq.github.io/code-of-conduct). We take it very |
| 5 | +seriously, and expect that you will as well. |
| 6 | + |
| 7 | +There are many ways you can contribute! :heart: |
| 8 | + |
| 9 | +## :bug: Bug Reports and Fixes |
| 10 | + |
| 11 | +- If you find a bug, please search for it in the |
| 12 | + [Issues](https://github.com/slackapi/python-slack-hooks/issues), and if it isn't |
| 13 | + already tracked, |
| 14 | + [create a new Bug Report Issue](https://github.com/slackapi/python-slack-hooks/issues/new/choose). |
| 15 | + Fill out the "Bug Report" section of the issue template. Even if an Issue is |
| 16 | + closed, feel free to comment and add details, it will still be reviewed. |
| 17 | +- Issues that have already been identified as a bug (note: able to reproduce) |
| 18 | + will be labelled `bug`. |
| 19 | +- If you'd like to submit a fix for a bug, |
| 20 | + [send a Pull Request](#creating-a-pull-request) and mention the Issue number. |
| 21 | +- Include tests that isolate the bug and verifies that it was fixed. |
| 22 | + |
| 23 | +## :bulb: New Features |
| 24 | + |
| 25 | +- If you'd like to add new functionality to this project, describe the problem |
| 26 | + you want to solve in a |
| 27 | + [new Feature Request Issue](https://github.com/slackapi/python-slack-hooks/issues/new/choose). |
| 28 | +- Issues that have been identified as a feature request will be labelled |
| 29 | + `enhancement`. |
| 30 | +- If you'd like to implement the new feature, please wait for feedback from the |
| 31 | + project maintainers before spending too much time writing the code. In some |
| 32 | + cases, `enhancement`s may not align well with the project objectives at the |
| 33 | + time. |
| 34 | + |
| 35 | +## :mag: Tests, :books: Documentation,:sparkles: Miscellaneous |
| 36 | + |
| 37 | +- If you'd like to improve the tests, you want to make the documentation |
| 38 | + clearer, you have an alternative implementation of something that may have |
| 39 | + advantages over the way its currently done, or you have any other change, we |
| 40 | + would be happy to hear about it! |
| 41 | +- If its a trivial change, go ahead and |
| 42 | + [send a Pull Request](#creating-a-pull-request) with the changes you have in |
| 43 | + mind. |
| 44 | +- If not, [open an Issue](https://github.com/slackapi/python-slack-hooks/issues/new) |
| 45 | + to discuss the idea first. |
| 46 | + |
| 47 | +If you're new to our project and looking for some way to make your first |
| 48 | +contribution, look for Issues labelled `good first contribution`. |
| 49 | + |
| 50 | +## Requirements |
| 51 | + |
| 52 | +For your contribution to be accepted: |
| 53 | + |
| 54 | +- [x] You must have signed the |
| 55 | + [Contributor License Agreement (CLA)](https://cla.salesforce.com/sign-cla). |
| 56 | +- [x] The test suite must be complete and pass. |
| 57 | +- [x] The changes must be approved by code review. |
| 58 | +- [x] Commits should be atomic and messages must be descriptive. Related issues |
| 59 | + should be mentioned by Issue number. |
| 60 | + |
| 61 | +If the contribution doesn't meet the above criteria, you may fail our automated |
| 62 | +checks or a maintainer will discuss it with you. You can continue to improve a |
| 63 | +Pull Request by adding commits to the branch from which the PR was created. |
| 64 | + |
| 65 | +[Interested in knowing more about about pull requests at Slack?](https://slack.engineering/on-empathy-pull-requests-979e4257d158#.awxtvmb2z) |
| 66 | + |
| 67 | +## Creating a Pull Request |
| 68 | + |
| 69 | +1. :fork_and_knife: Fork the repository on GitHub. |
| 70 | +2. :runner: Clone/fetch your fork to your local development machine. It's a good |
| 71 | + idea to run the tests just to make sure everything is in order. |
| 72 | +3. :herb: Create a new branch and check it out. |
| 73 | +4. :crystal_ball: Make your changes and commit them locally. Magic happens here! |
| 74 | +5. :arrow_heading_up: Push your new branch to your fork. (e.g. |
| 75 | + `git push username fix-issue-16`). |
| 76 | +6. :inbox_tray: Open a Pull Request on github.com from your new branch on your |
| 77 | + fork to `main` in this repository. |
| 78 | + |
| 79 | +## Maintainers |
| 80 | + |
| 81 | +There are more details about processes and workflow in the |
| 82 | +[Maintainer's Guide](https://github.com/slackapi/python-slack-hooks/blob/main/.github/maintainers_guide.md). |
0 commit comments