search

azureautomation / automation-packs

Collections of Automation resources that can include runbooks, configurations, modules, credentials, schedules, variables, connections, certificates, jobs, compilation jobs, and nodes.
MIT License
75 stars 42 forks source link

New remove resource groups automation pack #15

Closed dtzar closed 8 years ago

elcooper commented 8 years ago

Thanks for putting this together! I added a comment to this review in case you want to make the update, but this looks good.

dtzar commented 8 years ago

I'd like to keep the links as-is so if we update the underlying script / template it will be automatically reflected vs tying to a specific commit.

dtzar commented 8 years ago

Wasn't sure what to call it 101/102 etc with the naming here, but if it works for you - then all good :)

dtzar commented 8 years ago

One note - you might want to update the API version for some of your other templates using the 2015-01-01-preview versus the newer 2015-10-31 version.

elcooper commented 8 years ago

Thanks David! I’d been meaning to update the schema version, but just hadn’t gotten around to it yet. ☺

From: David Tesar [mailto:notifications@github.com] Sent: Tuesday, August 9, 2016 11:03 AM To: azureautomation/automation-packs automation-packs@noreply.github.com Cc: Beth Cooper elcooper@microsoft.com; State change state_change@noreply.github.com Subject: Re: [azureautomation/automation-packs] New remove resource groups automation pack (#15)

One note - you might want to update the API version for some of your other templates using the 2015-01-01-preview versus the newer 2015-10-31 version.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHubhttps://github.com/azureautomation/automation-packs/pull/15#issuecomment-238638165, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKUUYgUmiicp_iS0vbsmTjbyqRyjoR1lks5qeMDbgaJpZM4JfbiY.