Skip to content

Improve PR instructions for agents #8412

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.

Sign up for GitHub

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

Open
gonzaloriestra wants to merge 1 commit into
base: main
Choose a base branch
from
+10 −0
Open

Improve PR instructions for agents #8412

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ Fixes #0000 <!-- link to issue if one exists -->

<!--
Please, provide steps for the reviewer to test your changes locally.
Do not include here commands to run tests or other checks, automated CI will run them.
You can post a comment with `/snapit` to generate a snapshot of the changes to be tested.
-->

Expand Down
9 changes: 9 additions & 0 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,15 @@ Adhere to the following guidelines in your code:
- In tests, always avoid mocking the filesystem. Use real files and directories, in temporary directories if needed.
- In tests, prefer to have as little shared state between tests as possible. Avoid beforeAll and afterAll.

### PR creation

- Use GitHub stacks for multiple dependant PRs
- Follow the template from .github/PULL_REQUEST_TEMPLATE.md
- Be concise
- Remove empty sections and hidden comments
- Do not mark checklist items as completed (except the changelog one if added)
- In "How to test your changes?" only include CLI commands to test locally, do not add commands to run tests or other checks

## Changesets

Add a changeset only when the change is user-facing and ready to appear in public changelogs and release notes.
Expand Down
Loading