-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Replace the placeholder release build with our real one #10778
Conversation
Changes: - Convert all queue-time variables into form-based parameters - Add a "matrix" build strategy for Configs * Platforms - Rename all jobs to have reasonable names - The YAML generator has a bug where it inlines scripts *and* file paths if a task had both; remove old inlines - Remove dead rules
remove PR conditions
resourceFilePath: >- | ||
src\cascadia\TerminalApp\Resources\en-US\Resources.resw | ||
|
||
src\cascadia\TerminalControl\Resources\en-US\Resources.resw | ||
|
||
src\cascadia\TerminalConnection\Resources\en-US\Resources.resw | ||
|
||
src\cascadia\TerminalSettingsModel\Resources\en-US\Resources.resw | ||
|
||
src\cascadia\TerminalSettingsEditor\Resources\en-US\Resources.resw | ||
|
||
src\cascadia\WindowsTerminalUniversal\Resources\en-US\Resources.resw | ||
|
||
src\cascadia\CascadiaPackage\Resources\en-US\Resources.resw |
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.
We're not allowed to use wildcards here? Like, src\cascadia\*\Resources\en-US\Resources.resw
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.
no.
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.
we probably can, but i'd rather not mess with steps that are working
Hello @DHowett! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
This pull request ports our old release pipeline from Azure DevOps' editor to real YAML. It includes the following changes on top of a straight-up "export" from Azure: - Converts all queue-time variables into form-based parameters - Adds a "matrix" build strategy for Configs * Platforms - Renames all jobs to have reasonable names - The YAML generator has a bug where it inlines scripts *and* file paths if a task had both; remove old inlines - Removes dead rules - Fixes the WPF build to include the apiset impostor - Migrates the access token into the environment for the one build stage that needs it - Cleans up some of the online script logic - Removes all of the "!is pull request?" checks
This pull request ports our old release pipeline from Azure DevOps' editor to real YAML.
It includes the following changes on top of a straight-up "export" from Azure: