Several updates to deployThroughARM.ps1

[pcgeek86]

1. Updated string literals to use single quotes as best practice, to avoid accidental interpolation.
2. Removed unnecessary quotes around HashTable keys.
3. Added more relevant names to Resource Group and Automation Account names.

Cheers,
Trevor Sullivan
Microsoft MVP: PowerShell

[jodoglevy]

Hey @elcooper, in the future can you check with the owner before merging? This all looks fine except I don't like the new names of the resource group and automation account that are created. More than just AA DSC resources are created -- this is meant to be a sample environment for all Azure Automation features.

Trevor can you do another pull request where they're named "SampleAutomationAccount" and "SampleAutomationResourceGroup" or similar?

[elcooper]

Sure thing. Sorry about that Joe!

From: Joe Levy [mailto:[email protected]]
Sent: Monday, November 23, 2015 1:05 PM
To: azureautomation/automation-packs [email protected]
Cc: Beth Cooper [email protected]
Subject: Re: [automation-packs] Several updates to deployThroughARM.ps1 (#4)

Hey @elcooperhttps://github.com/elcooper, in the future can you check with the owner before merging? This all looks fine except I don't like the new names of the resource group and automation account that are created. More than just AA DSC resources are created -- this is meant to be a sample environment for all Azure Automation features.

Trevor can you do another pull request where they're named "SampleAutomationAccount" and "SampleAutomationResourceGroup" or similar?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-159063879.

[pcgeek86]

@jodoglevy Joe, I just went to update this, but it look like you already have done so yesterday.

3d358e1#diff-f1bbc499447edc8b9c3620ba2a1d404e

/cc @elcooper

Cheers,
Trevor Sullivan
Microsoft MVP: PowerShell

[jodoglevy]

Yup sorry forgot to update this thread
On Nov 25, 2015 6:13 PM, "Trevor Sullivan" [email protected] wrote:

@jodoglevy https://github.com/jodoglevy Joe, I just went to update
this, but it look like you already have done so yesterday.

3d358e1#diff-f1bbc499447edc8b9c3620ba2a1d404e
3d358e1#diff-f1bbc499447edc8b9c3620ba2a1d404e

/cc @elcooper https://github.com/elcooper

Cheers,
Trevor Sullivan
Microsoft MVP: PowerShell

—
Reply to this email directly or view it on GitHub
#4 (comment)
.