Skip to content

Conversation

@tuminoid
Copy link
Member

@tuminoid tuminoid commented Nov 5, 2025

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.

CLAUDE.md is added as symlink as Cladue STILL does not support AGENTS.md. It will be removed when Claude does. All the other CLIs do support it already.

@metal3-io-bot
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@metal3-io-bot metal3-io-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 5, 2025
@metal3-io-bot metal3-io-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Nov 5, 2025
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]>
@tuminoid tuminoid force-pushed the tuomo/add-copilot-instructions branch from 93308aa to e8acfa9 Compare December 9, 2025 10:50
@tuminoid tuminoid changed the title add copilot-instructions.md 📖 add AGENTS.md Dec 9, 2025
@tuminoid tuminoid marked this pull request as ready for review December 9, 2025 10:51
@metal3-io-bot metal3-io-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 9, 2025
@tuminoid
Copy link
Member Author

tuminoid commented Dec 9, 2025

Copy link
Member

@kashifest kashifest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

Copy link
Member

@lentzi90 lentzi90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Dec 9, 2025
@tuminoid
Copy link
Member Author

/approve

Can you switch your votes? @kashifest is reviewer and @lentzi90 is approver :)

@lentzi90
Copy link
Member

Sure!
/approve

@metal3-io-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kashifest, lentzi90

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 11, 2025
@lentzi90
Copy link
Member

"Translating" @kashifest approve -> lgtm

@metal3-io-bot metal3-io-bot merged commit 0f6fce5 into metal3-io:main Dec 11, 2025
14 of 15 checks passed
@metal3-io-bot metal3-io-bot deleted the tuomo/add-copilot-instructions branch December 11, 2025 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants