21245bf918
Signed-off-by: Daniel Thompson <daniel@redfelineninja.org.uk>
83 lines
2.9 KiB
ReStructuredText
83 lines
2.9 KiB
ReStructuredText
Contributor's Guide
|
|
===================
|
|
|
|
.. contents::
|
|
:local:
|
|
:depth: 2
|
|
|
|
Introduction
|
|
------------
|
|
|
|
Anyone can contribute to the wasp-os project. Contributions are typically made
|
|
via github using the typical fork-and-pull-request approach. Contributors who
|
|
do not wish to use github are welcome to share patches using ``git
|
|
format-patch --to wasp-os@redfelineninja.org.uk`` and ``git send-email``. In
|
|
both cases, the code will be reviewed by a project maintainer, so please
|
|
anticipate review comments and requests for changes. Typically pull
|
|
requests will not be merged if there are open questions or requests for
|
|
changes that have not been acted on.
|
|
|
|
All contributions must include a ``Signed-off-by`` tag added by the contributor
|
|
who submits the patch or patches. The ``Signed-off-by`` tag is added at the end
|
|
of the patch description and certifies that the contributor either wrote the
|
|
patch or has the right to share the code under the open source license
|
|
appropriate for the file being modified.
|
|
|
|
A ``Signed-off-by`` tag is an explicit statement that your contribution comes
|
|
under one of (a), (b), (c), or (d) from the list below so please be sure to
|
|
read carefully what you are certifying by adding your Signed-off-by.
|
|
|
|
Additionally please be aware that that contributors, like all other members of
|
|
the wasp-os community, are expected to meet the community guidelines described
|
|
in the project's code of conduct when interacting within all community spaces
|
|
(including the wasp-os github presence).
|
|
|
|
Coding Style
|
|
------------
|
|
|
|
wasp-os uses a similar coding style as micropython and, in particular, Python
|
|
code is expected to follow :pep:`8`.
|
|
|
|
Developer Certificate of Origin
|
|
-------------------------------
|
|
|
|
.. literalinclude:: dco.txt
|
|
:language: none
|
|
:emphasize-lines: 1-2,13
|
|
|
|
This procedure is the same one used by the Linux kernel project. To sign off a
|
|
patch append an appropriate line at the end of the commit message:
|
|
|
|
.. code-block:: none
|
|
|
|
Signed-off-by: Random Developer <r.developer@example.org>
|
|
|
|
Adding a sign-off can be automated by using git features such as
|
|
``git commit --signoff``. Please use your real name, anonymous and pseudonymous
|
|
contributions will not be accepted.
|
|
|
|
Git Hints and Tricks
|
|
--------------------
|
|
|
|
Quick fixes
|
|
~~~~~~~~~~~
|
|
|
|
The most common review feedback for contributions to wasp-os is a request that
|
|
the contributor include their sign-off. For a single patch at the head of the
|
|
current branch (and shared as a github pull request) this can be handled fairly
|
|
easily:
|
|
|
|
.. code-block:: sh
|
|
|
|
git commit --amend --signoff
|
|
git push <myfork> HEAD
|
|
|
|
Additionally, please be aware that github will not send out automatic
|
|
notifications to let the maintainer know that you have pushed an update to the
|
|
pull-request. Follow up the above with a comment on the pull request thread
|
|
saying that your contribution has been updated and is ready for another look.
|
|
|
|
Code of Conduct
|
|
---------------
|
|
|
|
.. include:: code_of_conduct.rst
|