Skip to content

Commit 8ac8580

Browse files
committed
updated based on comments to agents.md
On-behalf-of: @SAP mirza.kopic@sap.com Signed-off-by: Mirza Kopić <mirza.kopic@gmail.com>
1 parent c278d0f commit 8ac8580

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

AGENTS.md

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -78,19 +78,9 @@ Staged state machine: Unknown → Pending → InitialInProgress → InitialCompl
7878

7979
## Code Conventions
8080

81-
License header on all `.go` files — Apache-2.0, "The Platform Mesh Authors". Enforced by goheader linter.
81+
Tests: testify (require/assert). E2e: controller-runtime envtest.
8282

83-
Import order (enforced by gci):
84-
1. stdlib
85-
2. third-party
86-
3. `k8s.io/`
87-
4. `sigs.k8s.io/controller-runtime/`
88-
5. `sigs.k8s.io/multicluster-runtime/`
89-
6. `github.com/platform-mesh/`
90-
91-
Tests: testify (require/assert). E2e: controller-runtime envtest. All tests must be parallel (paralleltest linter).
92-
93-
Run `make codegen` after any change to `api/` types.
83+
Run `make codegen` after any change to `api/` types. Run `make lint-fix` before committing.
9484

9585
## Git
9686

@@ -102,6 +92,5 @@ Never add AI attribution to commits or PRs.
10292

10393
- Edit `zz_generated.deepcopy.go` — generated by controller-gen
10494
- Edit `config/*/crd/*.yaml` directly — generated by `make manifests`
105-
- Skip the license header — CI enforces it
10695
- Add tool-specific instruction files (`CLAUDE.md`, `.cursorrules`, etc.) — use this file
10796
- Commit without running `make codegen` after API type changes

0 commit comments

Comments
 (0)