From 561d84dae367f4eab2c2d6a70407977d256078d4 Mon Sep 17 00:00:00 2001 From: Aymeric Augustin Date: Mon, 28 Oct 2013 21:20:07 +0100 Subject: Extend the contribution docs. This is what I've understood from the projet over the last weeks. Fix #427. --- docs/contributing.rst | 42 +++++++++++++++++++++++++++++++++++------- 1 file changed, 35 insertions(+), 7 deletions(-) (limited to 'docs') diff --git a/docs/contributing.rst b/docs/contributing.rst index 061d9a3..56c8c09 100644 --- a/docs/contributing.rst +++ b/docs/contributing.rst @@ -1,6 +1,14 @@ Contributing ============ +Bug reports and feature requests +-------------------------------- + +You can report bugs and request features in the `bug tracker +`_. + +Please search the existing database for duplicates before filing an issue. + Code ---- @@ -14,18 +22,31 @@ the libraries required for working on the Debug Toolbar:: .. _virtualenv: http://www.virtualenv.org/ -Once you've done this, you can run the test suite on all supported version of -Django and Python:: +Tests +----- + +Once you've set up a development environment as explained above, you can run +the test suite:: + + $ make test + +You can also run the test suite on all supported versions of Django and +Python:: $ tox -Bug reports and feature requests --------------------------------- +This is strongly recommended before committing changes to Python code. -You can report bugs and request features in the `bug tracker -`_. +At this time, there isn't an easy way to test against databases other than +SQLite. The JaveScript code isn't tested either. -Please search the existing database for duplicates before filing an issue. +Style +----- + +Python code for the Django Debug Toolbar follows PEP8. Line length is limited +to 100 characters. You can check for style violations with:: + + $ make flake8 Patches ------- @@ -37,6 +58,10 @@ The Debug Toolbar includes a limited but growing test suite. If you fix a bug or add a feature code, please consider adding proper coverage in the test suite, especially if it has a chance for a regression. +If you change a CSS or a JavaScript file, you should update both the original +file and the minified version in the same commit. Use ``make compress_css`` +and ``make compress_js`` to minify files. + Translations ------------ @@ -45,6 +70,9 @@ Translation efforts are coordinated on `Transifex Help translate the Debug Toolbar in your language! +Prior to a release, the English ``.po`` file must be updated with ``make +translatable_strings``. Once translators have updated the translations on +Transifex, all ``.po`` files must be updated with ``make update_translations``. Mailing list ------------ -- cgit v1.2.3