clearlinux/graphene
2
0
Fork 0
mirror of https://github.com/clearlinux/graphene.git synced 2026-09-01 02:44:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

[Docs] Reformat CONTRIBUTING.rst to 80 columns, add Emacs setting

Browse Source
This commit is contained in:
Paweł Marczewski
2020-10-12 12:38:30 +02:00
committed by Dmitrii Kuvaiskii
parent 5c2f33ee49
commit 1b8e648ed8
2 changed files with 16 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.dir-locals.el
+1
View File
@@ -7,5 +7,6 @@
(asm-mode . ((indent-tabs-mode . t)))
(shell-script-mode . ((indent-tabs-mode . t)))
("\.git/COMMIT_EDITMSG" . ((nil . ((fill-column . 80)))))
(rst-mode . ((fill-column . 80)))
((nil . ((truncate-lines . t)))
(text-mode . ((eval . ((turn-on-auto-fill)))))))
CONTRIBUTING.rst
+15 -8
View File
@@ -39,7 +39,9 @@ Or you can see the archives at this google group:
https://groups.google.com/forum/#!forum/graphene-support
Please verify that your change doesn't introduce any insecure-by-default
functionality. If an option allows users to introduce a security risk, the option should have a name prefixed with ``insecure__`` and be disabled by default.
functionality. If an option allows users to introduce a security risk, the
option should have a name prefixed with ``insecure__`` and be disabled by
default.
Simple bugfixes need not have advance discussion, but we welcome queries from
newcomers.
@@ -175,8 +177,8 @@ For SGX, one needs to do the following::
If a |nbsp| test fails unexpectedly, one can use the :makevar:`KEEP_LOG=1`
option to get the complete output.
One can run tests manually (prepend the command with ``SGX=1`` or ``PAL_HOST=Linux-SGX`` to run the
SGX variant)::
One can run tests manually (prepend the command with ``SGX=1`` or
``PAL_HOST=Linux-SGX`` to run the SGX variant)::
/path/to/graphene/Scripts/run-pytest -v -rs test_pal.py
SGX=1 /path/to/graphene/Scripts/run-pytest -v -rs test_pal.py
@@ -186,8 +188,9 @@ It is also possible to run subset of tests::
/path/to/graphene/Scripts/run-pytest -v -rs test_pal.py::TC_01_Bootstrap
/path/to/graphene/Scripts/run-pytest -v -rs test_pal.py::TC_01_Bootstrap::test_100_basic_boostrapping
The ``run-pytest`` script is a wrapper for `pytest <https://docs.pytest.org/en/stable/usage.html>`__
and accepts the same command-line options.
The ``run-pytest`` script is a wrapper for `pytest
<https://docs.pytest.org/en/stable/usage.html>`__ and accepts the same
command-line options.
The shim unit tests work similarly, and are under
:file:`LibOS/shim/test/regression`.
@@ -226,6 +229,10 @@ The current members of the management team are:
The procedure for adding and removing maintainers
-------------------------------------------------
+ Joining: # of PRs submitted & merged + # of PRs reviewed + # of issues closed >= 20 (this means that a PR which fixes 3 issues counts as 4). Only complete and thorough reviews count.
+ Leaving: a member may be removed if not active or notoriously breaking rules from this document.
+ Additionally, at least 60% (rounded up) of current members have to agree to make any change to the team membership.
+ Joining: # of PRs submitted & merged + # of PRs reviewed + # of issues closed
>= 20 (this means that a PR which fixes 3 issues counts as 4). Only complete
and thorough reviews count.
+ Leaving: a member may be removed if not active or notoriously breaking rules
from this document.
+ Additionally, at least 60% (rounded up) of current members have to agree to
make any change to the team membership.