Repository navigation
NO-JIRA: Add CodeRabbit config to exclude autogenerated files #7905
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,61 @@ | ||
| inheritance: true | ||
| language: en-US | ||
| reviews: | ||
| profile: chill | ||
| high_level_summary: false | ||
| review_status: true | ||
| commit_status: true | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This and line 8 default to true already. |
||
| collapse_walkthrough: true | ||
| changed_files_summary: false | ||
| sequence_diagrams: false | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we not find value in this? |
||
| estimate_code_review_effort: false | ||
| poem: false | ||
| suggested_labels: false | ||
| path_filters: | ||
| # Vendor directories (managed by go mod vendor) | ||
| - "!vendor/**" | ||
| - "!**/vendor/**" | ||
|
|
||
| # Generated deepcopy, defaults, conversion, and other zz_generated files | ||
| - "!**/zz_generated*.go" | ||
| - "!**/zz_generated.crd-manifests/**" | ||
| - "!**/zz_generated.featuregated-crd-manifests/**" | ||
|
|
||
| # Generated CRD manifests and install assets | ||
| - "!cmd/install/assets/**/*.yaml" | ||
| - "!karpenter-operator/controllers/karpenter/assets/*.yaml" | ||
|
|
||
| # Generated client code (clientsets, informers, listers, apply configurations) | ||
| - "!client/**" | ||
|
|
||
| # Generated mock files | ||
| - "!**/*_mock.go" | ||
|
|
||
| # Generated protobuf files | ||
| - "!**/*.pb.go" | ||
| - "!**/*.pb.gw.go" | ||
| - "!**/generated.proto" | ||
|
|
||
| # Generated swagger doc files | ||
| - "!**/types_swagger_doc_generated.go" | ||
| - "!**/zz_generated.swagger_doc_generated.go" | ||
|
|
||
| # Generated API documentation | ||
| - "!docs/content/reference/api.md" | ||
| - "!docs/content/reference/aggregated-docs.md" | ||
|
|
||
| # Generated delegating client | ||
| - "!cmd/infra/aws/delegating_client.go" | ||
|
|
||
| # Generated stringer files | ||
| - "!**/*_string.go" | ||
| tools: | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should add gitleaks |
||
| golangci-lint: | ||
| enabled: true | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
| knowledge_base: | ||
| code_guidelines: | ||
| enabled: true | ||
| filePatterns: | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we not want to include the hypershift sme files here too? |
||
| - AGENTS.md | ||
| learnings: | ||
| scope: local | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we not find value in this?