-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Contribution guidelines
Gloin1313 edited this page Mar 1, 2017
·
7 revisions
- Respect PEP8 standards !
- Just the right amount of comments. Try to write auto-documented code (= with very explicit variable names).
- If you introduce a new functionality or fix a subtle bug, document it in the docstring/code.
- Keep the discussions on Github to their minimum. Think that many people receive these messages.
- Opinions are Ok.
- Avoid messages that don't bring the discussion forward ("thanks", etc.)
- For proper discussions, use the Moviepy Gitter. If these discussions lead to a decision (merge/reject), leave a message on the Moviepy issue that summarizes the reasons of the decision.
- Do not push any commit that changes the API without previous discussion
- Until we have a test suite, test the important changes with python 2 and 3 before merging to master