-
Notifications
You must be signed in to change notification settings - Fork 44.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* dx: Clean up PR template
- Loading branch information
Showing
1 changed file
with
32 additions
and
30 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,36 +1,38 @@ | ||
### Background | ||
|
||
<!-- Clearly explain the need for these changes: --> | ||
|
||
### Changes 🏗️ | ||
|
||
<!-- Concisely describe all of the changes made in this pull request: --> | ||
|
||
|
||
### Testing 🔍 | ||
> [!NOTE] | ||
Only for the new autogpt platform, currently in autogpt_platform/ | ||
|
||
<!-- | ||
Please make sure your changes have been tested and are in good working condition. | ||
Here is a list of our critical paths, if you need some inspiration on what and how to test: | ||
--> | ||
|
||
- Create from scratch and execute an agent with at least 3 blocks | ||
- Import an agent from file upload, and confirm it executes correctly | ||
- Upload agent to marketplace | ||
- Import an agent from marketplace and confirm it executes correctly | ||
- Edit an agent from monitor, and confirm it executes correctly | ||
|
||
### Configuration Changes 📝 | ||
> [!NOTE] | ||
Only for the new autogpt platform, currently in autogpt_platform/ | ||
|
||
If you're making configuration or infrastructure changes, please remember to check you've updated the related infrastructure code in the autogpt_platform/infra folder. | ||
|
||
Examples of such changes might include: | ||
|
||
- Changing ports | ||
- Adding new services that need to communicate with each other | ||
- Secrets or environment variable changes | ||
- New or infrastructure changes such as databases | ||
### Checklist 📋 | ||
|
||
#### For code changes: | ||
- [ ] I have clearly listed my changes in the PR description | ||
- [ ] I have made a test plan | ||
- [ ] I have tested my changes according to the test plan: | ||
<!-- Put your test plan here: --> | ||
- [ ] ... | ||
|
||
<details> | ||
<summary>Example test plan</summary> | ||
|
||
- [ ] Create from scratch and execute an agent with at least 3 blocks | ||
- [ ] Import an agent from file upload, and confirm it executes correctly | ||
- [ ] Upload agent to marketplace | ||
- [ ] Import an agent from marketplace and confirm it executes correctly | ||
- [ ] Edit an agent from monitor, and confirm it executes correctly | ||
</details> | ||
|
||
#### For configuration changes: | ||
- [ ] `.env.example` is updated or already compatible with my changes | ||
- [ ] `docker-compose.yml` is updated or already compatible with my changes | ||
- [ ] I have included a list of my configuration changes in the PR description (under **Changes**) | ||
|
||
<details> | ||
<summary>Examples of configuration changes</summary> | ||
|
||
- Changing ports | ||
- Adding new services that need to communicate with each other | ||
- Secrets or environment variable changes | ||
- New or infrastructure changes such as databases | ||
</details> |