Skip to content

Gemini review fix#197

Merged
vibhutikumar07 merged 6 commits intodevelopfrom
gemini-fix
Dec 11, 2025
Merged

Gemini review fix#197
vibhutikumar07 merged 6 commits intodevelopfrom
gemini-fix

Conversation

@vibhutikumar07
Copy link
Copy Markdown
Collaborator

Describe your changes

Any documentation

Type of change

Please find the below changes:

We have changed the model to 2.5flash from 1.5flash for Better result.
Changed the logic of review so that it'll be more specific toward PR changes
Now the review will happen on demand by commenting "gemini review"

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist before requesting a review

  • I have tested the functionality on my cloud environment.
  • I have provided sufficient automated/ unit tests for the code.
  • I have increased or maintained the test coverage.
  • I have ran integration tests on my cloud environment.
  • I have validated blackduck portal for any vulnerability after my commit.

Upload Screenshots/lists of the scenarios tested

  • I have Uploaded Screenshots or added lists of the scenarios tested in description

Comment thread .github/scripts/review.js
let responseBody = "Error: Could not generate a summary and recommendations for this issue.";
// --- END DETAILED PROMPT FIX ---

let recommendations = "Failed to generate recommendations.";
Comment thread .github/scripts/review.js Fixed
@vibhutikumar07 vibhutikumar07 merged commit 6c8354e into develop Dec 11, 2025
10 checks passed
@vibhutikumar07 vibhutikumar07 deleted the gemini-fix branch December 11, 2025 07:30
Juergen-Goebl-msggroup pushed a commit to Andreas-Meindl-msggroup/sdm that referenced this pull request Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants