Skip to content

Conversation

dmihalcik-virtru
Copy link
Member

  • The version information must be up to date and accurate before tagging and publishing releases

- The version information must be up to date and accurate before tagging and publishing releases
@dmihalcik-virtru dmihalcik-virtru requested a review from a team as a code owner July 1, 2025 17:11
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Summary of Changes

Hello @dmihalcik-virtru, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request refines the continuous integration process by upgrading the version validation script. The primary goal is to ensure that all relevant version information is consistently and accurately reflected across the codebase before any release is tagged and published, thereby preventing discrepancies and ensuring release integrity.

Highlights

  • Enhanced Version Detection Logic: The check-version-is.sh script now intelligently determines the expected version by parsing various inputs, including GitHub ref contexts (release branch names, tag names) and direct semantic version strings, improving its adaptability in CI/CD pipelines.
  • Improved Error Reporting Precision: The script's error messages for Makefile and lib/src/version.ts now use more precise line number detection by anchoring regexes to the start of the line, leading to more accurate feedback.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

sonarqubecloud bot commented Jul 1, 2025

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request introduces improvements to the version checking script, enhancing its ability to accurately validate version numbers across different files and contexts. The changes include parsing version information from release branches and tags, and refining the grep and sed commands for more precise version lookups. The review focuses on ensuring the sed commands are correctly anchored to prevent unintended matches.

@dmihalcik-virtru dmihalcik-virtru merged commit 0e97899 into main Jul 1, 2025
14 checks passed
@dmihalcik-virtru dmihalcik-virtru deleted the DSPX-1321-fix-sed-and-match-scripts branch July 1, 2025 20:58
@el-virt
Copy link
Contributor

el-virt commented Jul 1, 2025

/backport

@opentdf-automation
Copy link
Contributor

Created backport PR for release/sdk/v0.4:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-671-to-release/sdk/v0.4
git worktree add --checkout .worktree/backport-671-to-release/sdk/v0.4 backport-671-to-release/sdk/v0.4
cd .worktree/backport-671-to-release/sdk/v0.4
git reset --hard HEAD^
git cherry-pick -x 0e978991041beaab1ed006a3726d2cd0a69d881d
git push --force-with-lease

el-virt pushed a commit that referenced this pull request Jul 1, 2025
- The version information must be up to date and accurate before tagging and publishing releases

(cherry picked from commit 0e97899)
el-virt added a commit that referenced this pull request Jul 1, 2025
- The version information must be up to date and accurate before tagging and publishing releases

(cherry picked from commit 0e97899)

Co-authored-by: Dave Mihalcik <[email protected]>
Co-authored-by: el-virt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants