Skip to content

add initial, basic AGENTS.md#177

Merged
kcp-ci-bot merged 3 commits intokcp-dev:mainfrom
xrstf:agents
Apr 1, 2026
Merged

add initial, basic AGENTS.md#177
kcp-ci-bot merged 3 commits intokcp-dev:mainfrom
xrstf:agents

Conversation

@xrstf
Copy link
Copy Markdown
Contributor

@xrstf xrstf commented Mar 11, 2026

Summary

This is my first attempt at creating and committing a re-usable AI agent config file. It seems that AI usage is inevitable anyway and there are some things I would like to share with everyone else, like how e2e tests should be run.

I tried to follow https://agents.md/.

What Type of PR Is This?

/kind chore

Release Notes

NONE

On-behalf-of: @SAP christoph.mewes@sap.com
@kcp-ci-bot kcp-ci-bot added release-note-none Denotes a PR that doesn't merit a release note. kind/chore Categorizes issue or PR as related to maintenance and other usually non-code changes. dco-signoff: yes Indicates the PR's author has signed the DCO. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 11, 2026
@ntnn ntnn requested a review from Copilot March 11, 2026 21:39
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an initial AGENTS.md file to document shared guidance for AI agents and contributors, focusing on development workflow, SDK dependency handling, documentation conventions, testing, and changelog generation.

Changes:

  • Introduces AGENTS.md with recommended commands and contribution/testing guidelines.
  • Documents rules around SDK go.mod/Go version changes and changelog formatting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

On-behalf-of: @SAP christoph.mewes@sap.com
@xrstf xrstf requested review from Copilot and ntnn April 1, 2026 09:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@xrstf
Copy link
Copy Markdown
Contributor Author

xrstf commented Apr 1, 2026

/retest

On-behalf-of: @SAP christoph.mewes@sap.com
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ntnn ntnn added this to tbd Apr 1, 2026
@ntnn ntnn moved this to Reviewing in tbd Apr 1, 2026
Copy link
Copy Markdown
Member

@ntnn ntnn left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2026
@kcp-ci-bot
Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: d976170254280cd65dd9164d0f571061041e8c84

@xrstf
Copy link
Copy Markdown
Contributor Author

xrstf commented Apr 1, 2026

/approve

@kcp-ci-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ntnn, xrstf

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

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 1, 2026
@kcp-ci-bot kcp-ci-bot merged commit 88682c4 into kcp-dev:main Apr 1, 2026
10 checks passed
@github-project-automation github-project-automation bot moved this from Reviewing to Done in tbd Apr 1, 2026
@xrstf xrstf deleted the agents branch April 1, 2026 10:39
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. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/chore Categorizes issue or PR as related to maintenance and other usually non-code changes. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. 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