-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create pull_request_template.md #721
Conversation
📝 add PR template
add post-merge and ci jobs check
.github/pull_request_template.md
Outdated
- [ ] I have broken down my PR scope into the following TODO tasks | ||
- [ ] task 1 | ||
- [ ] task 2 | ||
- [ ] I have passed with at least one python version with `circleci local execute` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- I have run the tests locally and they passed. (here's how to use
circleci local execute
link???) - I have added tests for any new features or bugs I fixed
🐛 fix the contributing path
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@byronz I added one more checkbox to ask people to add tests (not just run the existing tests) - I'm happy with it, if you like my addition then 💃
Thanks for doing this!
📝 add PR template