-
Notifications
You must be signed in to change notification settings - Fork 40
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
This patch updates the documentation, fixing a few minor issues as well as providing security and contribution guides including project management informations.
- Loading branch information
Showing
4 changed files
with
117 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,96 @@ | ||
Contributing to pyCA | ||
==================== | ||
|
||
As an open source project, pyCA welcomes contributions of many forms. | ||
Examples of contributions include: | ||
|
||
- Code patches | ||
- Documentation improvements | ||
- Bug reports and patch reviews | ||
|
||
If you need more information about how to create a fork or a pull request, | ||
check out this `first contributions guide`_ or simply ask questions. | ||
|
||
|
||
Being Pragmatic | ||
--------------- | ||
|
||
For every rule, there is an exception. | ||
If you find that there is a good reason one of the following rules does not apply to you, | ||
please bring it up and explain why. | ||
We like to be pragmatic if necessary instead of blindly follow rules. | ||
|
||
|
||
Provide Necessary Information | ||
----------------------------- | ||
|
||
If you provide a patch, please also provide an explanation of the reasoning behind this patch. | ||
It is much easier to understand and review code if you know its intention upfront. | ||
If a pull request relates to an existing issue, please also link that issue. | ||
|
||
If you want to make us happy, please also provide this reasoning as part of your git commit messages. | ||
|
||
|
||
Tests | ||
----- | ||
|
||
PyCA comes with a set of tests which are run automatically on our CI system. | ||
Passing these tests is a requirement for all contributions. | ||
|
||
If the CI tests on your pull request fail and you are sure it is not caused by your patch, please complain. | ||
Errors happen and we can easily trigger a new build. | ||
Your patch cannot be merged without these tests passing. | ||
|
||
|
||
Documentation | ||
------------- | ||
|
||
If necessary, say for instance you modify the API, | ||
please also provide the documentation for your change as part of your pull request. | ||
Once a pull request is merged, the documentation should match that code. | ||
|
||
|
||
Reviews | ||
------- | ||
|
||
A reviewer will be assigned to your pull request to ensure that there are no issues. | ||
Once everything is fine, the reviewer will merge the pull request. | ||
Please communicate with the reviewer to address any issues. | ||
|
||
Please remember that reviewers are only human as well. | ||
|
||
|
||
Merging | ||
------- | ||
|
||
We prefer to `rebase and merge pull requests`_ if the contribution has a sane commit history | ||
or to `squash and merge your pull request commits`_ if they have not. | ||
This is meant to keep the overall commit history as clean as possible. | ||
If you prefer a specific merge mode for any reason, please indicate that on the pull request. | ||
|
||
|
||
Checklist | ||
--------- | ||
|
||
- Pull request `closes an accompanying issue`_ if one exists | ||
- Pull request has a proper title and description | ||
- Appropriate documentation is included | ||
- Code passes automatic tests | ||
- The pull request has a clean commit history | ||
- Commits have a `proper commit message`_ (title and body) | ||
|
||
|
||
Project Management | ||
------------------ | ||
|
||
This project was started and is watched over by `Lars Kiesow`_. | ||
Decisions are usually made as part of the community discussion and contributions are highly appreciated. | ||
But in case of conflicts, the project lead may make the final decision. | ||
|
||
|
||
.. _first contributions guide: https://github.com/firstcontributions/first-contributions#first-contributions | ||
.. _rebase and merge pull requests: https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-request-merges#rebase-and-merge-your-pull-request-commits | ||
.. _squash and merge your pull request commits: https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-request-merges#squash-and-merge-your-pull-request-commits | ||
.. _closes an accompanying issue: https://help.github.com/en/articles/closing-issues-using-keywords | ||
.. _proper commit message: https://chris.beams.io/posts/git-commit/ | ||
.. _Lars Kiesow: https://lkiesow.de |
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
Security Policy | ||
=============== | ||
|
||
Supported Versions | ||
------------------ | ||
|
||
We support only the latest version of this project. | ||
|
||
|
||
Reporting a Vulnerability | ||
------------------------- | ||
|
||
If you find a security vulnerability, | ||
please report it by sending a mail to [email protected]. | ||
We will discuss the problem internally and, if necessary, release a patched version as soon as possible. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters