-
Notifications
You must be signed in to change notification settings - Fork 21
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 support for BRM schema changes #258
Conversation
@BernieWhite any thoughts on this one? |
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.
Thanks for the contribution @jtracey93. Sorry this has taken a while to get to.
Mostly looks good, but double check my comments below. Otherwise all good to merge.
Also, I added some tests so we don't accidentally break this in the future. Double check you are ok this these.
Co-authored-by: Bernie White <bewhite@microsoft.com>
Co-authored-by: Bernie White <bewhite@microsoft.com>
@BernieWhite LGTM |
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.
Thanks @jtracey93 . All good to merge.
PR Summary
Add support for BRM schema changes as per Azure/bicep#10810
Metadata usage changes for Bicep Module Registry
summary
will becomedescription
to align with OCI registry (short summary/overview)description
(old) will becomedetails
(longer blurbs)PR Checklist