The aim of this document is to offer guidances for new contributors on how to contribute to raccoon.
For now, we only have a github repo where discussions are opened and welcomed. In a near future, we could come up with more communication vectors.
- Check if the bug hasn't yet been reported here.
- If no bug has been reported, please open an issue by using the issue template.
- Ensure the PR description describes the problem and the solution.
- Follow the commit messages template and don't forget to
add
Closes/Fixes #issue_number
to the commit message to link your patch to the corresponding issue.
- Check if a feature request isn't already opened.
- If not, suggest your change via an issue, here.
- Once issue participants are aligned, you can implement it.
TO LINK ONCE DEFINED.
We prefer a commit per fix with its own commit message to know what it corrects so that it can be selectively applied by a maintainer. We refuse large PR that makes multiple changes that have nothing to do with each other.
Crappy, complex, unreadable history is forbidden!
- We refuse merge commit in history. Instead use rebase.
- During a review, when you need to fix something, use fixup commit
- When the patch is validated, you'll be asked to
autosquash
your fixup commits.
TO DEFINE once test cases will be implemented.
Your patch will be discussed, as a patch submitter you are the owner of it until it gets merged in the main branch. It means that you have to answer questions about your change, fix nits/flaws that have been pointed out. If no activity or a lack of replies is reported on a PR, we will simply drop/delete it.
Make the patch against the most recent version possible.
Here is a short guide on how to write a commit message in raccoon.
[area]: [short line describing the intent of the patch]
-- empty line ---
[full description describing the aim of the patch, why this patch
is made]
-- empty line ---
[Closes/Fixes #issue_number]
[Reported-by: Name of the reporter]
The [area]
can be any part in the system, like doc
, strategy
, internal
. We don't
use conventional commits pattern.
Thanks.