Skip to content

RUST-130 Bump version to 1.5.0#215

Merged
sebastien-marichal merged 2 commits into
masterfrom
sma/bump-version
Mar 31, 2026
Merged

RUST-130 Bump version to 1.5.0#215
sebastien-marichal merged 2 commits into
masterfrom
sma/bump-version

Conversation

@sebastien-marichal

Copy link
Copy Markdown
Contributor

Epic: SKUNK-1662

Initial Prompt

Can create a new branch sma/bump-version based on origin/master?
Update the version (check the automated-release workflow onhow to do it)

Summary

Prepare the next development iteration by applying the version bump pattern from the automated release workflow.

Changes

  • Update the Gradle project versions to 1.5.0-SNAPSHOT
  • Update the analyzer crate version to 1.5.0 and regenerate analyzer/Cargo.lock

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Bump version to 1.5.0 RUST-130 Bump version to 1.5.0 Mar 31, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

RUST-130

@sebastien-marichal sebastien-marichal marked this pull request as ready for review March 31, 2026 07:49
@sebastien-marichal sebastien-marichal requested a review from a team March 31, 2026 07:49
@sonar-review-alpha

sonar-review-alpha Bot commented Mar 31, 2026

Copy link
Copy Markdown

Summary

Version bump for the next development iteration. Updates Gradle build versions from 1.4.0-SNAPSHOT to 1.5.0-SNAPSHOT across the project, and aligns the analyzer crate version from 0.1.0 to 1.5.0. Cargo.lock, license metadata, and about.toml are regenerated as part of the build process.

What reviewers should know

Files to review:

  • gradle.properties and sonar-rust-plugin/gradle.properties: Verify version is set to 1.5.0-SNAPSHOT
  • analyzer/Cargo.toml: Verify version is 1.5.0 (not -SNAPSHOT, matching analyzer releases)
  • analyzer/Cargo.lock: This is auto-regenerated; review to ensure no unexpected dependency changes

What NOT to worry about:

  • Dependency version updates in Cargo.lock are expected from cargo update
  • License file checksums changing is normal when dependencies update
  • about.toml checksum changes from regeneration

  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@ghislainpiot ghislainpiot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is bump-version action not working? It feels like it should be a better atlternative than manual PR

Comment thread analyzer/Cargo.lock
@sebastien-marichal

Copy link
Copy Markdown
Contributor Author

The Bump version job in the automated workflow has been skipped due to an issue with the reusable workflow. To avoid rerunning the full workflow I am doing the PR manually.

The Bump version depend on the unfreeze job which was skipped.

@ghislainpiot

ghislainpiot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

https://github.com/SonarSource/sonar-rust/pull/216/changes The action has a problem, the PR has a strange content. The CI does not run on the PR, so it might be hard to merge

@sonarqube-next

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed for 'sonar-rust'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@sonar-review-alpha sonar-review-alpha Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! ✅

Clean, mechanical version bump with no logic changes — safe to merge.

🗣️ Give feedback

@sebastien-marichal sebastien-marichal merged commit c8906e4 into master Mar 31, 2026
13 checks passed
@sebastien-marichal sebastien-marichal deleted the sma/bump-version branch March 31, 2026 08:29
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