config: enable CodeRabbit Multi-Repo Analysis#738
config: enable CodeRabbit Multi-Repo Analysis#738rsoaresd wants to merge 2 commits intocodeready-toolchain:masterfrom
Conversation
|
| inheritance: true | ||
| knowledge_base: | ||
| linked_repositories: | ||
| - repository: 'codeready-toolchain/toolchain-common' |
There was a problem hiding this comment.
It's a pity that we cannot add more than one linked repos. In this case I'm wondering if it's better to link toolchain-common or it's more critical to link host-operator. In theory they should not depend on each other but in practice there is some dependency between the two operators.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alexeykazakov, mfrancisc, rsoaresd The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
3 participants
Description
Enhance CodeRabbit code reviews to analyze changes across multiple linked repositories. For now, CodeRabbit Multi-Repo Analysis feature only allows one linked repository. Since host-operator depends on toolchain-common repo for "reusing" functions, we should link it.
Issue ticket number and link
SANDBOX-1677
Related PRs
codeready-toolchain/toolchain-common#521
codeready-toolchain/host-operator#1248
codeready-toolchain/registration-service#586
codeready-toolchain/toolchain-e2e#1267
kubesaw/ksctl#140
gc 189