diff options
author | IWASE Yusuke <iwase.yusuke0@gmail.com> | 2017-06-13 16:24:33 +0900 |
---|---|---|
committer | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2017-06-22 16:07:57 +0900 |
commit | 0f1430edfd80fb501f4f5324c84e55875e340a6d (patch) | |
tree | ba03a4f66f76f3b690e99d61c52ab432cf8d0e14 | |
parent | b4786a3fe3d4695804bfde83be4f44e771814f8c (diff) |
CONTRIBUTING: Update info of Python ver/lib and links
Signed-off-by: IWASE Yusuke <iwase.yusuke0@gmail.com>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
-rw-r--r-- | CONTRIBUTING.rst | 33 |
1 files changed, 20 insertions, 13 deletions
diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index 93e1890f..8e4945d3 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -37,28 +37,34 @@ features (it's not a must though). Python version and libraries ============================ -* Python 2.6+ - As RHEL 6 adopted python 2.6, features only for 2.7+ should be avoided. +* Python 2.7, 3.4, 3.5: -* standard library + widely used library - Basically widely used == OpenStack adopted - As usual there are exceptions. gevents. Or python binding library for other + Ryu supports multiple Python version. CI tests on Travis-CI is running + on these versions. + +* standard library + widely used library: + + Basically widely used == OpenStack adopted. + As usual there are exceptions. Or python binding library for other component. Coding style guide ================== -* pep8 +* pep8: + As python is used, PEP8 is would be hopefully mandatory for - http://www.python.org/dev/peps/pep-0008/ + https://www.python.org/dev/peps/pep-0008/ + +* pylint: -* pylint Although pylint is useful for finding bugs, but pylint score not very important for now because we're still at early development stage. + https://www.pylint.org/ -* Google python style guide is very helpful - http://google-styleguide.googlecode.com/svn/trunk/pyguide.html +* Google python style guide is very helpful: + http://google.github.io/styleguide/pyguide.html - Guidelines derived from Guido's Recommendations +* Guidelines derived from Guido's Recommendations: ============================= ================= ======== Type Public Internal @@ -76,10 +82,11 @@ Coding style guide Local Variables lower_with_under ============================= ================= ======== -* OpenStack Nova style guide +* OpenStack Nova style guide: https://github.com/openstack/nova/blob/master/HACKING.rst -* JSON files +* JSON files: + Ryu source tree has JSON files under ryu/tests/unit/ofproto/json. They are used by unit tests. To make patches easier to read, they are normalized using tools/normalize_json.py. Please re-run |