docs: add more ARM samples for JSON output and --initialPermissions flag#251
Merged
Conversation
bgdnext64
commented
Apr 9, 2026
Collaborator
- Add ARM JSON output example to installation-and-quickstart.md
- Add ARM --initialPermissions examples (comma-separated and @file.json) to installation-and-quickstart.md
- Add ARM --initialPermissions examples to commandline-flags-and-env-variables.md
- Fix --outputFormat json (non-existent flag) to --jsonOutput in display-options.MD
- Fix JSON output sample to match actual output format (flat map of scope to permissions)
- Add arm-initial-permissions.json sample file
Closed
Contributor
There was a problem hiding this comment.
Pull request overview
Updates MPF documentation to better cover ARM JSON output and the --initialPermissions flag, including correcting the documented JSON flag name and output shape, and adding a reusable sample permissions file.
Changes:
- Add new ARM quickstart examples for
--jsonOutputand--initialPermissions(comma-separated and@file.json) usage. - Fix docs to use
--jsonOutput(instead of the non-existent--outputFormat json) and update JSON output examples to the flat scope→permissions map format. - Add a sample
arm-initial-permissions.jsontemplate file.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
samples/templates/arm-initial-permissions.json |
Adds a sample JSON file for seeding initial permissions via --initialPermissions @file. |
go.mod |
Bumps the Go version directive. |
docs/installation-and-quickstart.md |
Adds ARM JSON output and --initialPermissions examples to the quickstart. |
docs/display-options.MD |
Fixes the JSON output flag name and updates the ARM JSON output example format. |
docs/commandline-flags-and-env-variables.md |
Adds ARM --initialPermissions examples (comma-separated and JSON file). |
maniSbindra
requested changes
Apr 22, 2026
- Add ARM JSON output example to installation-and-quickstart.md - Add ARM --initialPermissions examples (comma-separated and @file.json) to installation-and-quickstart.md - Add ARM --initialPermissions examples to commandline-flags-and-env-variables.md - Fix --outputFormat json (non-existent flag) to --jsonOutput in display-options.MD - Fix JSON output sample to match actual output format (flat map of scope to permissions) - Add arm-initial-permissions.json sample file
- Add language specifiers to fenced code blocks (MD040) - Add blank line before heading (MD022)
- Rename ARM JSON heading to 'ARM with Known Permissions (JSON File format)' - Quote @file paths in PowerShell --initialPermissions examples - Update Bicep JSON output description to flat scope->permissions map
44b9f1c to
3ee3f12
Compare
maniSbindra
approved these changes
Jun 9, 2026
maniSbindra
left a comment
Contributor
There was a problem hiding this comment.
accepted as remaining comments are for readability formatting in docs; not blocking
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.