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.

Sign up for GitHub

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

Jump to bottom

Update DeploymentScript to no longer use placeholders for parameters #42052

Closed

JoshLove-msft opened this issue Feb 18, 2024 · 0 comments

Assignees

Labels

Client Provisioning

Member

JoshLove-msft commented Feb 18, 2024

https://github.com/Azure/azure-sdk-for-net/blob/feature/cdk/sdk/provisioning/Azure.Provisioning/src/resources/DeploymentScript.cs#L65-L68 this was done for the initial hackathon and the serialization code knew how to unwrap these values. Now we have support for serializing Parameters into the bicep so we would need to essentially assign the parameters into the appropriate properties.

JoshLove-msft added Client Provisioning labels

JoshLove-msft mentioned this issue

Fix deploymentscript #42166

Merged

JoshLove-msft closed this as completed

JoshLove-msft self-assigned this

github-actions bot locked and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.