Skip to content

Conversation

@jessesimpson36
Copy link
Contributor

Which problem does the PR fix?

COPILOT_INSTRUCTIONS.md has never been followed for me on any AI agent I use whether it's github or opencode. This PR changes it to AGENTS.md so that it's agent-generic, and I also made it brief. COPILOT_INSTRUCTIONS was basically a "I am too lazy to make this myself so I'll have AI do it", and because I didn't know what to expect from it. But part of why it's not working well is because it's probably taking up too many tokens.

What's in this PR?

Checklist

Please make sure to follow our Contributing Guide.

Before opening the PR:

  • In the repo's root dir, run make go.update-golden-only.
  • There is no other open pull request for the same update/change.
  • Tests for charts are added (if needed).
  • In-repo documentation are updated (if needed).

After opening the PR:

  • Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
  • Did all checks/tests pass in the PR?

@jessesimpson36
Copy link
Contributor Author

I also added this line:

Complex logic for CI pipelines (>20 lines) should be implemented as golang scripts inside the scripts directory and then called with github actions. Do not implement this in bash.

Hopefully this will get us away from using bash by default just because we have a large legacy of bash

@jessesimpson36 jessesimpson36 marked this pull request as ready for review December 30, 2025 12:05
@jessesimpson36 jessesimpson36 requested a review from a team as a code owner December 30, 2025 12:05
@jessesimpson36 jessesimpson36 requested review from bkenez and eamonnmoloney and removed request for a team December 30, 2025 12:05
Signed-off-by: Jesse Simpson <jesse.simpson@camunda.com>
@jessesimpson36 jessesimpson36 merged commit 88be914 into main Jan 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants