Conversation
|
Because this actually makes K8up communicate that the use of an AI agent to contribute is OK, we should make sure all the maintainers agree on that. What's your take @k8up-io/maintainer ? |
Provide project structure, build commands, code conventions, PR rules, and testing guidance for AI coding assistants. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
Bug fixes must include a reproducing test before the fix. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
CI lints formatting — always run go fmt before committing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
When closing issues as already-implemented, ensure the feature is documented — undocumented features are invisible to users. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
Always link PRs to related issues so they're tracked and auto-closed on merge. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
Docs are AsciiDoc/Antora sources in this repo, built and deployed to docs.k8up.io via a webhook-triggered VSHN GitLab pipeline shared with other VSHN documentation repos. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
Always check git diff before committing and create PRs as draft when changes need user verification before maintainer review. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Aarno Aukia <aarno.aukia@vshn.ch>
|
We should add something similar to https://github.com/Sharkord/sharkord/blob/40cea8d607381ff1fef78fc3caf392698dda7a84/CONTRIBUTING.md?plain=1#L135
|
|
My opinion: I do not want to review low quality PRs nobody invested time in to understand the issue. Yes it's possible to fix simple stuff here and there but even that is very error prone (#1179 (comment), ...), complex issues are not understood by the agents (#1169), and the agents give completly wrong answers (#1009). Maintainer burnout is real and a growing security threat. Lets not add more onto our plate just because we need to follow every new, probably shortlived, hype. |
|
I agree with @bastjan. So far I've mostly ignored the flood of @arska's PRs. Even though most are only small changes, it's just too much and reviewing hallucinations might sometimes even take longer than implementing the change from scratch. Up until now all community contributions to K8up were very pleasant to work with and usually of reasonably high quality. Let's not set a bad precedence here. Having said that I'll try to go through the PRs today and review/merge/close them. |
Summary
AGENTS.mdto provide AI coding assistants (Claude Code, Copilot, Gemini CLI, etc.) with project context.editorconfig, and governance docsThis is a proposal for discussion — feedback welcome on what to include/exclude or adjust.
Test plan
🤖 Generated with Claude Code