Development Team/Code Review: Difference between revisions
| Line 17: | Line 17: | ||
== 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 | 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. | ||