-
Notifications
You must be signed in to change notification settings - Fork 168
Simplify the workflow triggers #146
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
Conversation
|
@gemini-cli /review |
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 changes simplify the workflow triggers and improve readability. I have one question about a potential behavior change in the pr-review workflow, but otherwise, the changes look good.
Carrying over changes in dogfooding: #144
|
@gemini-cli /review |
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.
Overall, the changes look good. The workflow triggers are now simpler and more secure. I have a few minor nits, but otherwise, this is ready to merge.
## What's Changed * Broken links by @kulaone in #129 * chore: mention user to confirm who run by @Marukome0743 in #131 * bug(#109): Support triggering PR review with issue comment by @leehagoodjames in #138 * feat(setup): enable setup script to automatically set repository variables by @leehagoodjames in #140 * Add `echo` to core tools for automated issue triage by @jerop in #143 * chore: simply GitHub Actions permissions by @sethvargo in #144 * Simplify the workflow triggers by @jerop in #146 * bug(/review): Fix agent not posting feedback in GitHub by @leehagoodjames in #141 ## New Contributors * @kulaone made their first contribution in #129 **Full Changelog**: v0.1.6...v0.1.7
Carrying over changes in dogfooding: #144
cc @sethvargo