[PR #3106] health: get rid of our hand-rolled sprintf->wstring helpers, prefer WIL #25243

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

Original Pull Request: https://github.com/microsoft/terminal/pull/3106

State: closed
Merged: Yes


Summary of the Pull Request

This is a general code health PR. I was reading about WIL, and found out that they've got these helpers already.

References

PR Checklist

  • Closes something (doesn't)
  • CLA signed
  • Tests added/passed
  • Requires documentation to be updated
  • WI_IsFlagSet(..., CoreContributor)

Validation

I ran the UUIDgen tests and used the azure cloud shell connector, and launched WT on the wrong architecture.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/3106 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request This is a general code health PR. I was reading about WIL, and found out that they've got these helpers already. ## References <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [ ] Closes something (doesn't) * [ ] CLA signed * [x] Tests added/passed * [ ] Requires documentation to be updated * [x] `WI_IsFlagSet(..., CoreContributor)` ## Validation I ran the UUIDgen tests and used the azure cloud shell connector, and launched WT on the wrong architecture.
claunia added the pull-request label 2026-01-31 09:08:14 +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#25243