This repository was archived by the owner on Feb 7, 2025. It is now read-only.
generated from CDCgov/template
-
Notifications
You must be signed in to change notification settings - Fork 10
OpenAI PR Agent Setup #1405
Closed
Closed
OpenAI PR Agent Setup #1405
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
592e947
added initial setup for openai pr agent
pluckyswan 8ec8364
remove .toml file and renamed env
pluckyswan 6c083eb
readd .toml file
pluckyswan e71443d
Add some env variables to pr_agent
saquino0827 b6a1359
change pr trigger on all PR updates
saquino0827 595737d
Adding every configuration combination possible
saquino0827 1ce9caa
Update API version
saquino0827 a8b1b54
Update PR agent name for testing purposes
saquino0827 68f0394
Update github actions in configurations
saquino0827 a6ddf8e
Testing next set of configurations
saquino0827 48906b1
Remove suggested invalid configurations
saquino0827 f425351
Update env variables to match our github secrets
saquino0827 0547def
Remove github settings from workflow and add to the .toml file
saquino0827 416ffe1
update settings
saquino0827 21e31f9
Add review settings to toml file
saquino0827 98813f4
Add preview to the azure version
saquino0827 add342d
add inline code comments and update comments
pluckyswan 4f5f36a
added comments for existing configuration and turned off labeling and…
pluckyswan b885e9b
specify type for `issue_comment`
somesylvie 8f18153
add additional trigger since `issue_comment` only works if workflow i…
somesylvie 4448436
let's see if we can stop labeling so much
somesylvie b80ee66
remove settings that are default; add a link to example settings
somesylvie c9f1a28
Merge branch 'main' into pr-agent
saquino0827 c0e5dba
adjust some more settings
somesylvie 94d1509
one more
somesylvie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add review settings to toml file
commit 21e31f99ce01204727cb47b2d8de9b7f54252016
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,3 +8,32 @@ auto_review = true | |
auto_describe = true | ||
auto_improve = true | ||
pr_actions = ["opened", "reopened", "ready_for_review", "review_requested"] | ||
|
||
[pr_reviewer] # /review # | ||
# enable/disable features | ||
require_score_review=false | ||
require_tests_review=true | ||
require_estimate_effort_to_review=true | ||
require_can_be_split_review=false | ||
require_security_review=true | ||
require_ticket_analysis_review=true | ||
# general options | ||
num_code_suggestions=0 | ||
inline_code_comments = false | ||
ask_and_reflect=false | ||
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. I'm curious to know what 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. 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. I think it'd be worth adding this option if there's no drawback to set that on |
||
#automatic_review=true | ||
persistent_comment=true | ||
extra_instructions = "" | ||
final_update_message = true | ||
# review labels | ||
enable_review_labels_security=true | ||
enable_review_labels_effort=true | ||
# specific configurations for incremental review (/review -i) | ||
require_all_thresholds_for_incremental_review=false | ||
minimal_commits_for_incremental_review=0 | ||
minimal_minutes_for_incremental_review=0 | ||
enable_intro_text=true | ||
enable_help_text=false # Determines whether to include help text in the PR review. Enabled by default. | ||
# auto approval | ||
enable_auto_approval=false | ||
maximal_review_effort=5 |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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'm curious about some of these settings - it might be helpful to either link out to docs or add comments explaining why non-obvious things are set certain ways (like this PR Agent actually suggests having a non-0 number for this setting, but I don't entirely understand what it does)
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.
That's a fair point. I'd have to see where they have docs for these settings because I only found a link to their example settings and not an explanation of the settings themselves.