Glenn Gailey

Results 36 comments of Glenn Gailey

Now that we have these commits in https://github.com/MicrosoftDocs/azure-docs-pr/pull/240246 we can #please-close this PR.

A more complete fixup was done in this PR in the private repo: https://github.com/MicrosoftDocs/azure-docs-pr/pull/244756 The PR includes your actual commits, so you will get credit. Thanks! #please-close

@adrianwyatt thanks for providing a fix here. It looks like someone has already merged your same fix in a different PR, so I won't be able to merge this one....

@BrunoLucasCloud this is a good idea. I've shortened your text a bit and I will merge your commit in a separate (internal) PR. Thanks for the good idea here!

Your commit is now in main in the private repo. You should see the updates this afternoon. Thanks! #please-close

Thanks @DevopsMercenary! We'll get this addressed soon.

@DevopsMercenary I cherry-picked the commits from this PR into my own PR in our private repo. Thanks for helping out—you should see your updates tomorrow after 10am PST. #please-close

@patelchandni from [the README](https://github.com/Azure/azure-quickstart-templates/blob/master/1-CONTRIBUTION-GUIDE/README.md#bicep-support): > 2. The azuredeploy.json must not be included in the PR as it will be built automatically when the sample is merged. Seems like the solution...

@patelchandni from [the README](https://github.com/Azure/azure-quickstart-templates/blob/master/1-CONTRIBUTION-GUIDE/README.md#bicep-support): > 2. The azuredeploy.json must not be included in the PR as it will be built automatically when the sample is merged. Seems like the solution...

Thanks @mike-urnun-msft, this is now fixed. #please-close