[PR #5256] [MERGED] Split rolling and PR builds. Drop ARM64, X86 from PR. #26205

Open
opened 2026-01-31 09:14:37 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5256
Author: @miniksa
Created: 4/6/2020
Status: Merged
Merged: 4/6/2020
Merged by: @undefined

Base: masterHead: dev/miniksa/builds


📝 Commits (4)

  • c647747 Separate rolling build and PR builds into their own defs. Drop ARM64 from PR as it has never found anything.
  • 6ea61ab collapse back into one def, add conditions.
  • c847249 Try with conditional stages.
  • e5e6677 Fix stage names, add display names.

📊 Changes

1 file changed (+39 additions, -17 deletions)

View changed files

📝 build/pipelines/ci.yml (+39 -17)

📄 Description

Summary of the Pull Request

Edits the definition file to distinguish further between the rolling build (the one that happens in master as it's updated) and the PR builds (that happen on every push to a pull request to master.) We will build less in PR since it rolls so often by removing the lines that reveal very few to no bugs at PR time. We'll leave them on in rolling so stuff can still be caught.

PR Checklist

  • Closes a desire to not waste builds.
  • I work here.
  • We'll see if the build still works.
  • No specific docs.
  • I talked about this with @DHowett-MSFT already.

Validation Steps Performed

  • This PR itself should validate that the definition still works in PRs. I think we have to wait for it to go to master to see if the trigger still works there.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/5256 **Author:** [@miniksa](https://github.com/miniksa) **Created:** 4/6/2020 **Status:** ✅ Merged **Merged:** 4/6/2020 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/miniksa/builds` --- ### 📝 Commits (4) - [`c647747`](https://github.com/microsoft/terminal/commit/c647747241961e52acc5b5fcf63879f8f2ce0372) Separate rolling build and PR builds into their own defs. Drop ARM64 from PR as it has never found anything. - [`6ea61ab`](https://github.com/microsoft/terminal/commit/6ea61abd94610b531d7163146f9b5de72fcb36bf) collapse back into one def, add conditions. - [`c847249`](https://github.com/microsoft/terminal/commit/c847249cf878fd719e6529341c8aa7f92d975427) Try with conditional stages. - [`e5e6677`](https://github.com/microsoft/terminal/commit/e5e6677c4a9e52e00876fcb780d3ccdcac30e755) Fix stage names, add display names. ### 📊 Changes **1 file changed** (+39 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `build/pipelines/ci.yml` (+39 -17) </details> ### 📄 Description ## Summary of the Pull Request Edits the definition file to distinguish further between the rolling build (the one that happens in master as it's updated) and the PR builds (that happen on every push to a pull request to master.) We will build less in PR since it rolls so often by removing the lines that reveal very few to no bugs at PR time. We'll leave them on in rolling so stuff can still be caught. ## PR Checklist * [x] Closes a desire to not waste builds. * [x] I work here. * [ ] We'll see if the build still works. * [x] No specific docs. * [x] I talked about this with @DHowett-MSFT already. ## Validation Steps Performed * [x] This PR itself should validate that the definition still works in PRs. I think we have to wait for it to go to master to see if the trigger still works there. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:14:37 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#26205