-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Contribution guidelines
Tom Burrows edited this page Feb 24, 2020
·
7 revisions
See CONTRIBUTING.md for instructions on how to set up the development environment locally.
- 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. Remember 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.