Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include all four version components in the cloud build number if three or more components are set in the version file #723

Merged
merged 1 commit into from Mar 17, 2022

Conversation

ap0llo
Copy link
Contributor

@ap0llo ap0llo commented Feb 17, 2022

As discussed in #401, this PR adapts calculation of the cloud build number to include all 4 version fields if either

  • The commit is configured to appear in the cloud build number as 4th version component (existing behavior) or
  • 3 version components are set in the version file and thus the git height is at the 4th position or
  • All 4 version components are set in the version file

…e or more components are set in the version file

If the version in version.json specifies three or four version components, include all four version fields in the cloud build number.
If three components are set in the version file, the git height is encoded as fourth component and should thus be included to make the cloud build number unique.
If four components are set, include the fourth component to ensure the cloud build number is at least as precise as the value in the version file.
Copy link
Collaborator

@AArnott AArnott left a comment

Choose a reason for hiding this comment

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

Thank you!

@AArnott AArnott added this to the v3.5 milestone Mar 17, 2022
@AArnott AArnott merged commit d857cf7 into dotnet:master Mar 17, 2022
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.

None yet

2 participants