Glenn Gailey
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