Skip to content

Minor YamlCreate bug Fixes#326440

Merged
denelon merged 3 commits intomicrosoft:masterfrom
Trenly:YamlFixDelete
Dec 26, 2025
Merged

Minor YamlCreate bug Fixes#326440
denelon merged 3 commits intomicrosoft:masterfrom
Trenly:YamlFixDelete

Conversation

@Trenly
Copy link
Copy Markdown
Contributor

@Trenly Trenly commented Dec 26, 2025

Checklist for Pull Requests

There's a small issue where with MSI files, the module responsible for fetching the MSI properties doesn't always release the file handle right away. Since the module is in the process of releasing the file handle, and there isn't great visibility into whether or not the file handle has been released, it makes the most sense just to retry the deletion.

This PR also fixes an issue where the ProductCode detection was broken after updating to use the PowerShell module

cc @denelon @mdanish-kh


Microsoft Reviewers: Open in CodeFlow

@Trenly Trenly requested a review from denelon as a code owner December 26, 2025 21:32
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 26, 2025
@Trenly Trenly changed the title Let file handles be released before deleting Minor YamlCreate bug Fixes Dec 26, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 26, 2025
@denelon denelon merged commit 6acf5d5 into microsoft:master Dec 26, 2025
2 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Dec 26, 2025
@Trenly Trenly deleted the YamlFixDelete branch December 27, 2025 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Moderator-Approved One of the Moderators has reviewed and approved this PR Project-File

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants