-
Notifications
You must be signed in to change notification settings - Fork 88
chore: initial Qodo configuration #2181
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
|
||
| [rag_arguments] | ||
| enable_rag=true | ||
| rag_repo_list=['redhat-developer/rhdh','redhat-developer/red-hat-developers-documentation-rhdh'] No newline at end of file |
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.
Any other repositories that are worth including in RAG for context?
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.
I think this should suffice for now, better to start small and add more context as necessary rather than ending up with irrelevant observations from other repos. For future reference, do the repositories in this list need to also have Qodo enabled and configured?
|
| commitable_code_suggestions = false | ||
|
|
||
| [config] | ||
| ignore_pr_authors = ["renovate", "rhdh-bot", "dependabot"] |
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.
Skip for those bot PRs?
| @@ -0,0 +1,38 @@ | |||
| [jira] | |||
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.
There is also an option to use a Wiki for storing and periodically updating the best practices. We don't have it yet enabled for any other repository, but it can make sense to use it.
Please look here:
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.
The COPE team was discussing last week how Qodo would be useful in helping to review new plugin requests as there is a standard set of items to be checked for these PRs. I think this could be placed in a custom best_practices.md file, but it seems like this can work in conjunction with the autogenerated one that is enabled with the Wiki. I can add the custom file in a subsequent PR and also look into adding the Wiki + autogenerated best practices.
hopehadfield
left a comment
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.
LGTM for getting it started! Thanks for driving this @zdrapela



Hey, I just made a Pull Request!
Configuration can be found in Qodo docs:
✔️ Checklist