Skip to content

Fix: Update Gemini model version in presubmit.yml#3339

Open
prem-2006 wants to merge 3 commits intoiluwatar:masterfrom
prem-2006:prem-2006-patch-1
Open

Fix: Update Gemini model version in presubmit.yml#3339
prem-2006 wants to merge 3 commits intoiluwatar:masterfrom
prem-2006:prem-2006-patch-1

Conversation

@prem-2006
Copy link

Summary

Fixes the failing Presubmit.ai check caused by an outdated Gemini model version.

Changes Made

  • Updated LLM_MODEL from gemini-1.5-flash to gemini-1.5-flash-latest in .github/workflows/presubmit.yml.

Related Issue

Fixes #3337

The Presubmit.ai check was failing due to an outdated Gemini model version.
Updated LLM_MODEL from "gemini-1.5-flash" to "gemini-1.5-flash-latest" to fix the issue.
@github-actions
Copy link

github-actions bot commented Oct 12, 2025

PR Summary

Fixes a failing Presubmit.ai check by updating Gemini model version in the presubmit workflow. Updates the PR trigger types, removes an unnecessary secret check, switches to latest checkout action, and prints a completion message.

Changes

File Summary
.github/workflows/presubmit.yml Changed the presubmit workflow to use actions/checkout@v4, run AI reviewer with LLM_MODEL set to gemini-1.5-flash-latest and LLM_PROVIDER ai-sdk, removed the secret check step, broadened PR triggers to include reopened and ready_for_review, and added a final completion message.

autogenerated by presubmit.ai

@sonarqubecloud
Copy link

Updated pull request event types and removed secret check step. Modified AI reviewer step to include new environment variables.
@github-actions
Copy link

This PR is stale because it has been open 60 days with no activity.

@github-actions github-actions bot added the Stale label Dec 15, 2025
Refactor presubmit workflow for AI code review
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.

🚨 Pull request needs attention.

Review Summary

Commits Considered (3)
  • e8288fe: Merge pull request #1 from prem-2006/prem-2006-patch-2

Refactor presubmit workflow for AI code review

  • 16eeea7: Refactor presubmit workflow for AI code review

Updated pull request event types and removed secret check step. Modified AI reviewer step to include new environment variables.

  • fc7bd0c: fix: update Gemini model version in presubmit.yml (#3337)

The Presubmit.ai check was failing due to an outdated Gemini model version.
Updated LLM_MODEL from "gemini-1.5-flash" to "gemini-1.5-flash-latest" to fix the issue.

Files Processed (1)
  • .github/workflows/presubmit.yml (1 hunk)
Actionable Comments (1)
  • .github/workflows/presubmit.yml [20-26]

    possible_bug: "Ensure YAML env mapping is valid"

Skipped Comments (3)
  • .github/workflows/presubmit.yml [10-10]

    enhancement: "Update PR event types"

  • .github/workflows/presubmit.yml [17-18]

    best_practice: "Add repository checkout step"

  • .github/workflows/presubmit.yml [28-29]

    best_practice: "Completion message"

Comment on lines +20 to +26
- name: Run AI code reviewer
uses: presubmit/ai-reviewer@latest
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
LLM_API_KEY: ${{ secrets.LLM_API_KEY }}
LLM_MODEL: "gemini-1.5-flash"
LLM_PROVIDER: ai-sdk
LLM_MODEL: gemini-1.5-flash-latest # ✅ Updated model
REVIEW_SUMMARY: "true"
MAX_TOKENS: "4096"
Copy link

Choose a reason for hiding this comment

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

This block configures the AI reviewer task. However, ensure the env block is properly formed under the step (use env: with a mapping of variables). The snippet currently shows an 'env' line with a possible formatting issue (env + LLM_PROVIDER...). Correct it to:\n env:\n LLM_PROVIDER: ai-sdk\n LLM_MODEL: gemini-1.5-flash-latest\n REVIEW_SUMMARY: "true"\n MAX_TOKENS: "4096"\n TEMPERATURE: "0.2"

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue: Presubmit Check Failure

1 participant