Skip to content

PARENTOSS-117 use BUILD_NUMBER#271

Merged
julien-carsique-sonarsource merged 1 commit intomasterfrom
fix/jcarsique/PARENTOSS-117-buildNumber
Jul 17, 2025
Merged

PARENTOSS-117 use BUILD_NUMBER#271
julien-carsique-sonarsource merged 1 commit intomasterfrom
fix/jcarsique/PARENTOSS-117-buildNumber

Conversation

@julien-carsique-sonarsource
Copy link
Copy Markdown
Contributor

@julien-carsique-sonarsource julien-carsique-sonarsource commented Jul 17, 2025

PARENTOSS-117
Use BUILD_NUMBER first for buildInfo.buildNumber and deployProperties.build.number, before other fallback variables
Use GITHUB_SHA first for deployProperties.vcs.revision and buildInfo.vcsRevision
Use GITHUB_REF first for deployProperties.vcs.branch

Also collect environment variables with patterns: GITHUB_SHA, GITHUB_REF, GITHUB_REPOSITORY, GITHUB_RUN*
Sort envVarsIncludePatterns

Use BUILD_NUMBER first for buildInfo.buildNumber and deployProperties.build.number, before other fallback variables
Use GITHUB_SHA first for deployProperties.vcs.revision and buildInfo.vcsRevision
Use GITHUB_REF first for deployProperties.vcs.branch

Also collect environment variables with patterns: GITHUB_SHA, GITHUB_*REF*, GITHUB_REPOSITORY, GITHUB_RUN*
Sort envVarsIncludePatterns
Copilot AI review requested due to automatic review settings July 17, 2025 16:07
@julien-carsique-sonarsource julien-carsique-sonarsource requested a review from a team as a code owner July 17, 2025 16:07
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 Maven configuration to prioritize GitHub Actions environment variables in the build and deployment process. The changes ensure that GitHub-specific environment variables are used first before falling back to other CI/CD platform variables.

  • Reorders environment variable patterns to be alphabetically sorted
  • Adds GitHub-specific environment variable patterns (GITHUB_REF, GITHUB_REPOSITORY, GITHUB_RUN*)
  • Prioritizes BUILD_NUMBER, GITHUB_SHA, and GITHUB_REF in deployment properties and build info configurations

@julien-carsique-sonarsource julien-carsique-sonarsource enabled auto-merge (rebase) July 17, 2025 16:08
@sonarqube-next
Copy link
Copy Markdown

@julien-carsique-sonarsource julien-carsique-sonarsource merged commit 2425322 into master Jul 17, 2025
5 checks passed
@julien-carsique-sonarsource julien-carsique-sonarsource deleted the fix/jcarsique/PARENTOSS-117-buildNumber branch July 17, 2025 16:14
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