From 5fcc6578c5701efd1fd22f10f52d0e9057f2523a Mon Sep 17 00:00:00 2001 From: merwanehamadi Date: Tue, 5 Sep 2023 20:45:45 -0700 Subject: [PATCH] Update PULL_REQUEST_TEMPLATE.md --- .github/PULL_REQUEST_TEMPLATE.md | 52 ++++++++++++++++++++++++++------ 1 file changed, 42 insertions(+), 10 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index e56d7c5b..efb67868 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,17 +1,49 @@ -### Background + - + + +### Background + ### Changes - -### PR Quality Checklist +### Documentation + +### Test Plan + + +### PR Quality Checklist +- [ ] My pull request is atomic and focuses on a single change. +- [ ] I have thoroughly tested my changes with multiple different prompts. +- [ ] I have considered potential risks and mitigations for my changes. +- [ ] I have documented my changes clearly and comprehensively. +- [ ] I have not snuck in any "extra" small tweaks changes. - [ ] I have run the following commands against my code to ensure it passes our linters: - ```shell - black . --exclude test.py - isort . - mypy . - autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring --in-place agbenchmark - ``` + ```shell + black . + isort . + mypy + autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests --in-place + ``` + + + +