- Follow the C++ Core Guidelines cpp core guideline
- Try to follow PEP guidelines and google python style guide
- Try to follow the SOLID principles of clean code
- Use meaningful variable names
- For most important commits use commit message template
-
Open issue with full feature description
-
Wait for response from core team
-
Open an issue
-
Create new branch and code there
-
Commit your work using commit message template
-
Be sure that you are not behind the master, if so, rebase your branch
-
Make a pull request to master branch with meaningful description and sign the CLA which is part of the PR checks
-
Wait for review
-
Apply / argue the code review comments
-
Clean up the comment history if needed
-
Push the changes again
-
Repeat steps 4-10 until convergence
Side note: DO NOT USE git merge master
or git pull master
to align your
branch with master, use git rebase master
for it!