You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+2-1Lines changed: 2 additions & 1 deletion
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -28,7 +28,8 @@ Here are some important resources:
28
28
## Pull Request Guidelines:
29
29
* Please name your pull request appropriately as it will become the commit message.
30
30
* Please leave a meaningful description on the Pull Request so the reviewers have better context when reviewing.
31
-
* Please assign reviewers once the CI checks have passed.
31
+
* Please assign reviewers once the CI checks have passed. + Please also tag the reviewers in a message on the general slack
32
+
* PRs must be reviewed by another member of the team before being checked by the maintainer team
32
33
* Please use github commenting for code review discussions and resolve the issues before approving the pull request and merging it.
33
34
* All pull requests must include counterpart tests, as long as the folder structure above is honored CI will automatically run them. Test can also be run locally using pytest
34
35
* Commits should be squashed when merging a PR and no merge commits should be created.
0 commit comments