-
Notifications
You must be signed in to change notification settings - Fork 564
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
Update PR Review and Description Generation to Use YAML #188
Conversation
PR Analysis
PR Feedback
|
da7f693
to
fe75e3f
Compare
|
||
Example output: | ||
```yaml | ||
PR Title: ... |
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.
Maybe try
PR Title: |-
...
Preparing review... |
PR Analysis
PR Feedback
|
Preparing review... |
Update PR Review and Description Generation to Use YAML
PR Type:
Enhancement, Tests
PR Description:
This PR updates the PR review and description generation process to use YAML instead of JSON. It introduces new utility functions to load and fix YAML data, and modifies the existing functions and classes to use these new utilities. The PR also updates the prompts for PR description and review to use YAML schema, and adds unit tests for the new YAML loading function.
Additional change:
the prompt now uses 'Previous title', 'Previous description' and 'Commit messages' . It has the following message to accompany them:
Notice that the 'Previous title', 'Previous description' and 'Commit messages' sections may be partial, simplistic, non-informative or not up-to-date. Hence, compare them to the PR diff code, and use them only as a reference.
@zmeirPR Main Files Walkthrough:
pr_agent/algo/utils.py
: Added new functions to load and fix YAML data.pr_agent/tools/pr_description.py
: Updated the PR description preparation process to load data from YAML instead of JSON.pr_agent/tools/pr_reviewer.py
: Updated the PR review preparation process to load data from YAML instead of JSON.tests/unittest/test_load_yaml.py
: Added unit tests for the new YAML loading function.pr_agent/settings/pr_description_prompts.toml
: Updated the PR description prompt to use YAML schema.pr_agent/settings/pr_reviewer_prompts.toml
: Updated the PR reviewer prompt to use YAML schema.