Make sure we print env vars for JSON ARM templates #10749
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the environment-output logic to add explicit handling for Bicep templates and consolidates the Windows/.NET branch.
- Introduces an
$isBicepflag for Bicep template detection - Merges the
.envoutput conditions into singleif/elseifstatements - Adds contextual comments for .NET encryption and Bicep handling
Comments suppressed due to low confidence (1)
eng/common/TestResources/TestResources-Helpers.ps1:267
- Variable
$isBicepshould be PascalCased (e.g.$IsBicep) to align with PowerShell variable naming conventions used elsewhere ($OutFile,$IsWindows).
$isBicep = $templateFile.originalFilePath -and $templateFile.originalFilePath.EndsWith(".bicep")
|
The following pipelines have been queued for testing: |
|
Hi @heaths. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days. |
github-actions
bot
added
the
no-recent-activity
github-actions
bot
removed
the
no-recent-activity
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#10749 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Heath Stewart <[email protected]>
|
Merged this because eng/common sync had already completed. |
5 participants
Fixes #10734