-
Notifications
You must be signed in to change notification settings - Fork 18
Open
Description
Define the requirements for merging code into the main.
This includes requirements of the code. For example:
- Tests included
- Documentation included
- Define minimal coverage
- Docstrings available
- Example available as notebook
And it includes the reviewing process
- Who are suitable as reviewer?
- Code can be run by reviewer
- All code, examples and tests are reviewed for logic
- All code, examples and tests are reviewed for readability
- Is a review step by Van Oord needed?
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Todo