Skip to content

BUILD-9047 fix version pattern check#104

Merged
tomverin merged 1 commit intomasterfrom
BUILD-9047
Sep 12, 2025
Merged

BUILD-9047 fix version pattern check#104
tomverin merged 1 commit intomasterfrom
BUILD-9047

Conversation

@tomverin
Copy link
Copy Markdown
Contributor

No description provided.

@tomverin tomverin requested a review from a team as a code owner September 12, 2025 11:50
Copilot AI review requested due to automatic review settings September 12, 2025 11:50
Copy link
Copy Markdown

Copilot AI 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 Overview

This PR updates the version pattern validation in the releasability status check to support more flexible version formats, including project names, milestone versions, and different build number separators.

  • Updated regex pattern to allow optional project prefixes, milestone markers, and multiple build number separator formats
  • Enhanced error messages with more comprehensive examples and clearer format description
  • Removed input validation for optional_checks parameter

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
releasability-status/find_version.sh Updated version pattern regex and error messages to support extended version formats
releasability-status/action.yml Removed validation logic for optional_checks input parameter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sonarqubecloud
Copy link
Copy Markdown

🤖 Pull Request summary

Relaxes version format validation to support more complex versioning schemes.

• Removes input validation for optional_checks parameter in action.yml
• Updates version regex pattern to accept project prefixes, milestone suffixes, and build numbers
• Improves error messaging with clearer format examples and expectations

Review focus: Verify the new regex pattern correctly handles all intended version formats while maintaining security against injection attacks.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

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

@tomverin tomverin enabled auto-merge (squash) September 12, 2025 11:57
@tomverin tomverin merged commit fde5613 into master Sep 12, 2025
7 checks passed
@tomverin tomverin deleted the BUILD-9047 branch September 12, 2025 12:07
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.

3 participants