Skip to content

Conversation

@felipebbc
Copy link
Contributor

@felipebbc felipebbc commented Mar 20, 2021

Description

Added a conditional during Bicep CLI installation to send install messages to logger for an "auto install" while still printing messages to stdout for a manual install (az bicep install)

Testing Guide

History Notes

[ARM] Fix #17379: bicep auto install results in invalid json output from deployment


This checklist is used to make sure that common guidelines for a pull request are followed.

@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Mar 20, 2021
@ghost
Copy link

ghost commented Mar 20, 2021

Thank you for your contribution felipebbc! We will review the pull request and get back to you soon.

@yonzhan yonzhan requested review from jsntcy and zhoxing-ms March 20, 2021 23:55
@yonzhan yonzhan added this to the S185 milestone Mar 20, 2021
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 20, 2021

ARM

@felipebbc felipebbc marked this pull request as ready for review March 21, 2021 03:05
@felipebbc felipebbc requested a review from Juliehzl as a code owner March 21, 2021 03:05
@zhoxing-ms zhoxing-ms merged commit 212f925 into Azure:dev Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bicep auto install results in invalid json output from deployment

3 participants