While we want to encourage you to submit code contributions and patches to Savane, we do need to make sure they are cleanly implemented, make sense in the way they are designed and are consistent with the rest of the software.

Firstly, read README when it exists in the directory you are working. Almost every recommandations for specific parts of the code are in README. For instance, if you modify a CSS file, read the css/README.

Is this legal? You absolutely need to add copyright and licence details in each file you include.

Is this scalable? For example, if you want to have a pop-up box listing groups or users, that obviously would never scale. It might work great if you have 10 projects on your test version of savannah, but what happens when you have 10,000?

Is it commented? Did you add simple comments throughout your code explaining why/how you're approaching the task?

When you prepare your patches, make sure they are tested and known to work, and make sure that each patch handles basically one feature at a time. If you group a bunch of features into a single patch, we may not be able to accept them all, or all at the same time.

Copyright (C) 2004-2006, the Gna! people. Posted items are owned by whoever posted them.
Verbatim copying and distribution of this entire article is permitted in any medium, provided this notice is preserved.