-
Notifications
You must be signed in to change notification settings - Fork 517
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
Update Invoke-PolicyToBicep.ps1
Scripts to Handle -
's,
's (spaces) & '
apostrophes + Fix #254
#288
Conversation
additional Task woulb be: escape |
Hey @SeSeicht could you provide some more explanation or screenshots of what you mean here as I can't work it out 👍 |
|
Invoke-PolicyToBicep.ps1
Scripts to Handle -
's and
's (spaces)Invoke-PolicyToBicep.ps1
Scripts to Handle -
's,
's (spaces) & '
apostrophes
adding parameters
…eicht/PolicytoBicepScriptParams
add more params
Invoke-PolicyToBicep.ps1
Scripts to Handle -
's,
's (spaces) & '
apostrophesInvoke-PolicyToBicep.ps1
Scripts to Handle -
's,
's (spaces) & '
apostrophes + Fix #254
/azp run validateazcloud |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
Other than the initial lines at the top of the scripts I have not comments
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
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.
LGTM
Overview/Summary
Update
's (spaces) &
Invoke-PolicyToBicep.ps1
Scripts to Handle-
's,'
apostrophes + Fix #254This PR fixes/adds/changes/removes
Invoke-PolicyToBicep.ps1
Scripts to Handle-
's,'
apostrophesBreaking Changes
None.
Testing Evidence
Local testing with ISD colleague has proved successful
As part of this Pull Request I have
main
branch