Skip to content

BUILD-8678: set build number in actions#37

Merged
hedinasr merged 1 commit intomasterfrom
feat/hnasr/BUILD-8678-setBuildNumberInActions
Aug 5, 2025
Merged

BUILD-8678: set build number in actions#37
hedinasr merged 1 commit intomasterfrom
feat/hnasr/BUILD-8678-setBuildNumberInActions

Conversation

@hedinasr
Copy link
Copy Markdown
Contributor

@hedinasr hedinasr commented Aug 4, 2025

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 4, 2025

🤖 Pull Request summary

Updates build actions to use centralized build number generation.

• Adds get-build-number step to all build actions (Gradle, Maven, NPM, Poetry, Yarn)
• Removes commented-out build number logic from Poetry action
• Standardizes build number handling across all build tools

Review focus: Verify that existing build processes still reference the correct build number ID and that the centralized action provides the expected outputs.

💬 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

@hedinasr hedinasr marked this pull request as ready for review August 5, 2025 06:06
@hedinasr hedinasr requested review from a team and Copilot August 5, 2025 06:06
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 implements setting build numbers in GitHub Actions by adding a standardized get-build-number step to all build action workflows. The change ensures consistent build number generation across different build systems (Maven, Gradle, NPM, Poetry, and Yarn).

  • Adds a "Get build number" step to all build action configurations
  • Removes commented-out code from the Poetry build action
  • Standardizes the build number generation process across all build tools

Reviewed Changes

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

Show a summary per file
File Description
build-yarn/action.yml Adds get-build-number step before build parameters setup
build-poetry/action.yml Adds get-build-number step and removes commented-out legacy code
build-npm/action.yml Adds get-build-number step before build parameters setup
build-maven/action.yml Adds get-build-number step before build parameters setup
build-gradle/action.yml Adds get-build-number step before build parameters setup

Comment thread build-yarn/action.yml
Comment thread build-poetry/action.yml
Comment thread build-npm/action.yml
Comment thread build-maven/action.yml
Comment thread build-gradle/action.yml
@hedinasr hedinasr merged commit 746c5b3 into master Aug 5, 2025
12 checks passed
@hedinasr hedinasr deleted the feat/hnasr/BUILD-8678-setBuildNumberInActions branch August 5, 2025 06:16
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