-
Notifications
You must be signed in to change notification settings - Fork 16
Contributor Guidelines
Kevin Chang edited this page Jan 13, 2021
·
3 revisions
- Docstrings for methods
- Minimum amount of print statements when possible.
- Descriptive commits (PR number - changes)
12 - Fixed abc bug
- Descriptive method names
- Explain how to replicate changes. Before/after behavior is ideal
- No giant pull requests, split up by feature
- Minor changelog in description helps reviewers