1,157
edits
Changes
Development Team/Code Review (view source)
Revision as of 21:06, 23 December 2008
, 21:06, 23 December 2008→Patch guidelines
== Patch guidelines ==
== Patch guidelines ==
Please try to use [http://www.logilab.org/857 pylint] to verify your patch for things like exceeding 80 columns etc, unused imports and unused variables. Pylint is by far a tool you can 100% rely on but it helps to follow some guidelines and to avoid the most stupid errors like typos. Here is a pylintrc you can use: [http://dev.laptop.org/git?p=sugar-jhbuild;a=blob_plain;f=scripts/data/pylintrc pylintrc]
Please try to use [http://www.logilab.org/857 pylint] to verify your patch for things like exceeding 80 columns etc, unused imports and unused variables. Pylint is not a tool you can rely on 100% but it helps to follow some guidelines and to avoid the most stupid errors like typos. Here is a pylintrc you can use: [http://dev.laptop.org/git?p=sugar-jhbuild;a=blob_plain;f=scripts/data/pylintrc pylintrc]
In the sugar packages use 'make distcheck' to make sure all files are included and the POTFILES.in is up to date.
In the sugar packages use 'make distcheck' to make sure all files are included and the POTFILES.in is up to date.