-
Notifications
You must be signed in to change notification settings - Fork 625
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
How best_practices.md is being used? #1288
Comments
Also, I hope to understand the effect of adding instructions in the content of best_practices section in .pr_agent.toml vs extra_instructions in pr_review section in .pr_agent.toml. Thank you so much! |
best_practice file is a feature of pr-agent pro (there is a "💎" sign). |
@mrT23 thank you for the answer! Also, I am observing review quality being different for e.g. java code vs IOS code. If I understand correctly, the default model was finetuned with 25k PR's, and I am wondering what these 25k PR's are composed of (distribution of PR's across programming languages). It also mentions that the pro version can configure it to use solely claude or gpt model here, are these models finetuned as well? What are the models we can experiment with in the config file other than 'claude-3-5-sonnet' and 'gpt-4o'. Thanks again |
|
Hello, I am trying to understand how best_practices.md is being used in this document section
Is this relevant_best_practices here retrieving info from best_practices.md file and add to the LLM context? I am not seeing where best_practices.md is being used in the code.
Thank you!
The text was updated successfully, but these errors were encountered: