Skip to content

Conversation

@lqiu96
Copy link
Member

@lqiu96 lqiu96 commented Aug 11, 2025

@product-auto-label product-auto-label bot added the size: xl Pull request size is extra large. label Aug 11, 2025
@lqiu96 lqiu96 requested review from blakeli0 and zhumin8 August 11, 2025 14:53
@lqiu96 lqiu96 marked this pull request as ready for review August 11, 2025 14:53
@lqiu96 lqiu96 enabled auto-merge (squash) August 11, 2025 14:54
Copy link
Contributor

@blakeli0 blakeli0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to try out this on a branch first?

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

@lqiu96 lqiu96 closed this Aug 11, 2025
auto-merge was automatically disabled August 11, 2025 16:31

Pull request was closed

@lqiu96 lqiu96 reopened this Aug 11, 2025
@github-actions
Copy link

There is a problem with the Gemini CLI PR review. Please check the action logs for details.

1 similar comment
@github-actions
Copy link

There is a problem with the Gemini CLI PR review. Please check the action logs for details.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Review Summary

This pull request introduces a set of GitHub Actions workflows to integrate the Gemini CLI for various tasks like pull request reviews and issue triage. The workflows are well-structured and make good use of GitHub Actions features.

🔍 General Feedback

  • The use of google-github-actions/run-gemini-cli is consistent across all workflows.
  • The prompts for the Gemini CLI are well-defined and provide good context for the AI.
  • The workflows are triggered on appropriate events.

I have left one comment regarding a block of commented-out code that should be removed. Otherwise, the changes look good.

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

Copy link
Member Author

@lqiu96 lqiu96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gemini-cli /review

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

Ah, it seems like the reason this is skipping is because I am CONTRIBUTOR and not COLLABORATOR.

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

Ah, it seems like the reason this is skipping is because I am CONTRIBUTOR and not COLLABORATOR.

Hmm, I am a member of googleapis org. Not sure why this workflow isn't triggering. Going to set my status as public for the googleapis org.

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

@lqiu96 lqiu96 closed this Aug 11, 2025
@lqiu96 lqiu96 reopened this Aug 11, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Review Summary

This pull request introduces a set of GitHub Actions workflows to integrate the Gemini CLI for various tasks like automated issue triage, PR reviews, and general interaction. The workflows are well-structured and seem to be generated from a template, following good practices.

🔍 General Feedback

  • The workflows are comprehensive and cover a good range of use cases for the Gemini CLI.
  • The use of a GitHub App for authentication is a good security practice.
  • The prompts provided to the Gemini CLI are detailed and well-thought-out.
  • I've suggested a minor improvement to use the gh CLI consistently for posting comments, which aligns with the existing style in the workflows.

Overall, this is a great addition to the repository's automation capabilities.

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

1 similar comment
@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

@lqiu96 lqiu96 closed this Aug 11, 2025
@lqiu96 lqiu96 reopened this Aug 11, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Review Summary

This PR introduces four new GitHub workflows to integrate the Gemini CLI for various automated tasks. The workflows are for general CLI interaction, automated and scheduled issue triage, and pull request reviews.

🔍 General Feedback

  • The workflows are well-structured and make good use of GitHub Actions features like concurrency control and reusable steps.
  • There are a few areas for improvement, mainly related to the correctness of action references and the simplification of conditional logic.
  • Pinning actions to a specific commit hash is a good security practice.
  • The if condition for the PR review workflow was commented out, which would have caused it to run on every PR event. This has been flagged for correction.

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

@product-auto-label product-auto-label bot removed the size: xl Pull request size is extra large. label Aug 11, 2025
@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

1 similar comment
@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

Copy link
Member Author

@lqiu96 lqiu96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gemini-cli /review

@lqiu96 lqiu96 closed this Aug 11, 2025
@lqiu96 lqiu96 reopened this Aug 11, 2025
@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli /review

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@gemini-cli explain this PR

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📋 Review Summary

This PR introduces two new GitHub workflows for interacting with the Gemini CLI. The workflows are well-structured, but there are a few areas that need improvement, primarily around security, stability, and consistency.

🔍 General Feedback

  • Security: The gemini-cli.yml workflow is missing a step to generate a GitHub token, which will cause it to fail.
  • Stability: The actions used in both workflows should be pinned to specific versions to avoid unexpected changes.
  • Consistency: The versions of actions/checkout and actions/github-script should be consistent with the recommended versions.
  • Clarity: The commit messages in gemini-cli.yml could be more descriptive.
  • Correctness: The if condition in gemini-pr-review.yml is commented out, which will cause the workflow to run on every pull request and issue comment. This should be uncommented.

Overall, the changes are good, but addressing the feedback will make the workflows more robust and maintainable.

@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

Oh... I'm trying to invoke this command via a comment from a branch. The docs say: This event will only trigger a workflow run if the workflow file exists on the default branch. (https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#issue_comment). We would only be able to use @gemini-cli /review comment once this is merged into main.

@lqiu96 lqiu96 enabled auto-merge (squash) August 11, 2025 19:10
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Aug 11, 2025
@lqiu96 lqiu96 force-pushed the use-gemini-cli-gh-actions branch from 2c1dbc6 to 904c6f9 Compare August 11, 2025 19:13
@product-auto-label product-auto-label bot added size: l Pull request size is large. and removed size: xl Pull request size is extra large. labels Aug 11, 2025
@lqiu96 lqiu96 requested a review from blakeli0 August 11, 2025 19:16
@lqiu96
Copy link
Member Author

lqiu96 commented Aug 11, 2025

@blakeli0 @zhumin8 PTAL again. Gemini CLI should be able to be invoked via when pull request is created or when @gemini-cli /review is invoked from a comment.

@lqiu96 lqiu96 closed this Aug 12, 2025
auto-merge was automatically disabled August 12, 2025 16:15

Pull request was closed

@lqiu96 lqiu96 reopened this Aug 12, 2025
@lqiu96 lqiu96 enabled auto-merge (squash) August 12, 2025 16:15
@sonarqubecloud
Copy link

@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@lqiu96 lqiu96 merged commit 00195e6 into main Aug 12, 2025
101 checks passed
@lqiu96 lqiu96 deleted the use-gemini-cli-gh-actions branch August 12, 2025 16:52
lqiu96 added a commit that referenced this pull request Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants