-
Notifications
You must be signed in to change notification settings - Fork 519
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add installation of ALZ Powershell module for policy scripts #679
Conversation
@oZakari these scripts are invoked by https://github.com/Azure/ALZ-Bicep/blob/main/.github/workflows/update-policy.yml & https://github.com/Azure/ALZ-Bicep/blob/main/.github/workflows/update-policy-china.yml, do we need to install the ALZ module as part of these workflows? Or is that already covered? |
Thanks @jtracey93 for linking those, I've updated the MC workflow and added some logging as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple of tweaks
Co-authored-by: Jack Tracey <41163455+jtracey93@users.noreply.github.com>
Co-authored-by: Jack Tracey <41163455+jtracey93@users.noreply.github.com>
Overview/Summary
Now that the Edit-LineEndings function has been moved to the ALZ PowerShell Module, I have adjusted the Invoke-PolicytoBicep scripts to prompt users to install the module.
#678
This PR fixes/adds/changes/removes
Breaking Changes
Testing Evidence
Performs test to ensure it prompts the user prior to installing and that it wouldn't prompt user again after the initial installation.
As part of this Pull Request I have
.bicep
file/s I am adding/editing are using the latest API version possiblemain
branch