-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[release/8.0] Add support for hotfix branding #65252
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
base: release/8.0
Are you sure you want to change the base?
Conversation
|
Hi @@mmitche. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document. |
There was a problem hiding this 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 pull request adds support for "hotfix" as an additional pre-release label for ASP.NET Core 8.0 servicing builds, enabling the build system to recognize hotfix builds alongside regular servicing builds.
Changes:
- Modified the
IsServicingBuildcondition to accept both "servicing" and "hotfix" as valid pre-release labels
| <AspNetCoreMajorMinorVersion>$(AspNetCoreMajorVersion).$(AspNetCoreMinorVersion)</AspNetCoreMajorMinorVersion> | ||
| <!-- Servicing builds have different characteristics for the way dependencies, baselines, and versions are handled. --> | ||
| <IsServicingBuild Condition=" '$(PreReleaseVersionLabel)' == 'servicing' ">true</IsServicingBuild> | ||
| <IsServicingBuild Condition=" '$(PreReleaseVersionLabel)' == 'servicing' or '$(PreReleaseVersionLabel)' == 'hotfix'">true</IsServicingBuild> |
Copilot
AI
Jan 28, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space before the closing quote and angle bracket. For consistency with other conditions in this file (see line 25), there should be a space before the closing ">.
| <IsServicingBuild Condition=" '$(PreReleaseVersionLabel)' == 'servicing' or '$(PreReleaseVersionLabel)' == 'hotfix'">true</IsServicingBuild> | |
| <IsServicingBuild Condition=" '$(PreReleaseVersionLabel)' == 'servicing' or '$(PreReleaseVersionLabel)' == 'hotfix' ">true</IsServicingBuild> |
Add support for the pre-release label to be "hotfix" for 8.0