-
Notifications
You must be signed in to change notification settings - Fork 0
1.5. PR Checklist
Gabe edited this page Aug 24, 2024
·
2 revisions
- You'll want to make a draft PR as soon as possible so that others can see you're working & so you can ask questions by referring to your changes (github comments) & mentioning one of the maintainers.
- When you receive a comment from someone else, then make the change they requested, don't resolve the question yourself. Instead, respond with the commit you made the change in & the original requester will resolve the comment.
-
Did you link your issue?
-
Did you comment possibly confusing parts of your changes?
-
Did you test that all the functions your changes affected seem to work?
-
If you updated a class, did you update any properties that may depend on that change?
-
If you modified a database table, did you create alembic migrations & test using the main database?
a. Did you merge any alembic conflicts?
-
Did you request a review from a maintainer?