[PR #2179] [MERGED] Try to turn audit mode back on without building test/utilities #24845

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2179
Author: @miniksa
Created: 7/31/2019
Status: Merged
Merged: 8/1/2019
Merged by: @DHowett-MSFT

Base: masterHead: dev/miniksa/audit


📝 Commits (3)

  • f61e060 Attempt to remove all test and utility projects from audit mode (and turn it back on) to see if that keeps it within the disk space boundaries.
  • 23ef8d7 drop x86 and arm configs for the test projects too.
  • 9e46dc3 @zadjii-msft, a.k.a. captain doesn't actually generate guids, strikes again. fixed one I missed.

📊 Changes

2 files changed (+3 additions, -68 deletions)

View changed files

📝 OpenConsole.sln (+0 -63)
📝 build/pipelines/ci.yml (+3 -5)

📄 Description

Summary of the Pull Request

Audit mode was turned off because it ran out of disk. We talked to the Azure Pipelines Managed Agents team. They gave us some ideas. Then I had one too: don't audit test libraries and utilities and maybe we'll save enough space to get through this.

PR Checklist

Validation Steps Performed

This happens on build agents so this PR is the validation.


🔄 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/2179 **Author:** [@miniksa](https://github.com/miniksa) **Created:** 7/31/2019 **Status:** ✅ Merged **Merged:** 8/1/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/miniksa/audit` --- ### 📝 Commits (3) - [`f61e060`](https://github.com/microsoft/terminal/commit/f61e06033e338c09436b023f8cf14ecba869669b) Attempt to remove all test and utility projects from audit mode (and turn it back on) to see if that keeps it within the disk space boundaries. - [`23ef8d7`](https://github.com/microsoft/terminal/commit/23ef8d7be95ce0a2178c31d37d5d20d4bfa07b51) drop x86 and arm configs for the test projects too. - [`9e46dc3`](https://github.com/microsoft/terminal/commit/9e46dc307f3ee2eff6324a068704fb889c9fd658) @zadjii-msft, a.k.a. captain doesn't actually generate guids, strikes again. fixed one I missed. ### 📊 Changes **2 files changed** (+3 additions, -68 deletions) <details> <summary>View changed files</summary> 📝 `OpenConsole.sln` (+0 -63) 📝 `build/pipelines/ci.yml` (+3 -5) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Audit mode was turned off because it ran out of disk. We talked to the Azure Pipelines Managed Agents team. They gave us some ideas. Then I had one too: don't audit test libraries and utilities and maybe we'll save enough space to get through this. <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [x] Closes #2098 * [x] I'm an employee * [x] This is a test * [x] Doesn't need documentation * [x] I am a core contributor <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed This happens on build agents so this PR *is* the validation. --- <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:05:41 +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#24845