Skip to content

Add comments to explain gRPC fields#18863

Merged
anthony-c-martin merged 1 commit intomainfrom
ant/grpcdoc
Jan 20, 2026
Merged

Add comments to explain gRPC fields#18863
anthony-c-martin merged 1 commit intomainfrom
ant/grpcdoc

Conversation

@anthony-c-martin
Copy link
Member

@anthony-c-martin anthony-c-martin commented Jan 20, 2026 •
edited
Loading

Mostly generated using copilot prompt:

Copy the comments from https://github.com/Azure/bicep-extensibility/blob/main/docs/vnext/openapi.yaml

@anthony-c-martin anthony-c-martin mentioned this pull request Jan 20, 2026
Merged
@github-actions
Copy link
Contributor

github-actions bot commented Jan 20, 2026 •
edited
Loading

Test this change out locally with the following install scripts (Action run 21183742754)

VSCode
  • Mac/Linux 
    bash <(curl -Ls https://aka.ms/bicep/nightly-vsix.sh) --run-id 21183742754
  • Windows 
    iex "& { $(irm https://aka.ms/bicep/nightly-vsix.ps1) } -RunId 21183742754"
Azure CLI
  • Mac/Linux 
    bash <(curl -Ls https://aka.ms/bicep/nightly-cli.sh) --run-id 21183742754
  • Windows 
    iex "& { $(irm https://aka.ms/bicep/nightly-cli.ps1) } -RunId 21183742754"

andyleejordan
andyleejordan approved these changes Jan 20, 2026
View reviewed changes
Copy link
Contributor

@andyleejordan andyleejordan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@anthony-c-martin anthony-c-martin merged commit d4b28b6 into main Jan 20, 2026
40 of 41 checks passed
@anthony-c-martin anthony-c-martin deleted the ant/grpcdoc branch January 20, 2026 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeskew jeskew jeskew approved these changes

+1 more reviewer

@andyleejordan andyleejordan andyleejordan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

📘 Docs Not Needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@anthony-c-martin @jeskew @andyleejordan