-
Notifications
You must be signed in to change notification settings - Fork 121
📖 add AGENTS.md #2917
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.
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
base: main
Are you sure you want to change the base?
📖 add AGENTS.md #2917
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
d66f170 to
0a889e9
Compare
0a889e9 to
309198c
Compare
|
/hold |
0ea0026 to
c232bd0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds comprehensive documentation for AI coding agents working on the Cluster API Provider Metal3 (CAPM3) project. The new AGENTS.md file serves as a detailed reference guide that can be used by AI assistants (including GitHub Copilot and other AI coding tools) to better understand the project's architecture, conventions, and workflows.
Key changes:
- Comprehensive 1,488-line AGENTS.md file documenting project structure, development workflows, testing guidelines, and coding conventions
- .github/copilot-instructions.md file that references AGENTS.md using the @ syntax
- .gitignore entry for .claude directory to exclude AI assistant configuration files
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| AGENTS.md | New comprehensive AI agent documentation covering architecture, Makefile targets, CI/CD, code patterns, testing guidelines, E2E testing details, and behavioral guidelines |
| .gitignore | Adds exclusion for .claude directory used by AI assistant configurations |
| .github/copilot-instructions.md | References AGENTS.md to provide context to GitHub Copilot |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
0c4b00c to
655805a
Compare
Add AGENTS.md to allow AI assistants to be more effective, especially the Code Review Bot. Note: do not create .github/copilot-instructions.md as it will prevent current version of Copilot CLI from reading AGENTS.md at all. Signed-off-by: Tuomo Tanskanen <[email protected]>
655805a to
23e11f2
Compare
|
I wonder if the Copilot review would pick it up from PR, now that the copilot-instructions.md mess is cleared up. |
|
/cc @lentzi90 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Add AGENTS.md to allow AI assistants to be more effective, especially the Code Review Bot.
Note: do not create .github/copilot-instructions.md as it will prevent current version of Copilot CLI from reading AGENTS.md at all.