Skip to content

Tailor CodeRabbit configuration for crossplane-runtime#875

Merged
negz merged 2 commits intocrossplane:mainfrom
negz:rabbits
Oct 10, 2025
Merged

Tailor CodeRabbit configuration for crossplane-runtime#875
negz merged 2 commits intocrossplane:mainfrom
negz:rabbits

Conversation

@negz
Copy link
Member

@negz negz commented Oct 10, 2025

Description of your changes

The CodeRabbit configuration was copied from the main Crossplane repository, which is an application with CLI commands, Helm charts, and E2E tests. This repository is a Go library with none of those things.

This PR removes path instructions for cmd, cluster, design, test/e2e, and internal/controller directories that don't exist here. It adds pkg/reconciler path instructions instead. The breaking change check now applies to all public Go code since this is a library where all exported APIs are public. The feature gate check is removed - feature flags are implemented by library consumers, not in the library itself.

I have:

negz added 2 commits October 10, 2025 12:42
Signed-off-by: Nic Cope <nicc@rk0n.org>
The CodeRabbit configuration was copied from the main Crossplane repository,
which is an application with CLI commands, Helm charts, and E2E tests. This
repository is a Go library with none of those things.

This commit removes path instructions for cmd, cluster, design, test/e2e,
and internal/controller directories that don't exist here. It adds
pkg/reconciler path instructions instead. The breaking change check now
applies to all public Go code since this is a library where all exported
APIs are public. The feature gate check is removed - feature flags are
implemented by library consumers, not in the library itself.

Signed-off-by: Nic Cope <nicc@rk0n.org>
@negz negz requested a review from a team as a code owner October 10, 2025 19:52
@negz negz requested a review from jbw976 October 10, 2025 19:52
@negz negz merged commit 3f03019 into crossplane:main Oct 10, 2025
10 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.

2 participants