-
Notifications
You must be signed in to change notification settings - Fork 10
Wiki update dec #31
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
Wiki update dec #31
Conversation
Updated script calls to include './' for execution. Clarified instructions for multiple subscriptions.
Added instructions for generating reports for multiple regions.
Updated script execution commands to include './' for local execution.
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.
Pull request overview
This pull request updates the Azure Region Selection documentation to improve consistency and clarity for PowerShell script usage and multi-region/multi-subscription scenarios.
Key Changes:
- Updated PowerShell script examples to consistently use the
.\prefix for script execution - Added comprehensive multi-region documentation with explicit instructions for generating per-region reports
- Improved multi-subscription documentation by specifying
subscriptions.jsonas the required filename and correcting the parameter syntax
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| docs/wiki/Step-by-Step-Guide.md | Removed trailing whitespace from PowerShell command example |
| docs/wiki/3-CostInformation.md | Changed heading from "Example" to "Multiple Regions Example" for clarity |
| docs/wiki/2-AvailabilityCheck.md | Added new "Multiple Regions" section with instructions and updated examples with clearer region name formatting |
| docs/wiki/1-Collect.md | Updated all script examples to use .\ prefix and improved multi-subscription instructions with specific filename and corrected parameter syntax |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
jfaurskov
left a comment
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
This pull request updates documentation to improve clarity and consistency for running PowerShell scripts and handling multi-region and multi-subscription scenarios. The most important changes are grouped below:
Script Usage Consistency and Clarity
docs/wiki/1-Collect.mdto use the `.This pull request updates documentation to improve clarity and consistency for running PowerShell scripts and handling multi-region and multi-subscription scenarios. The most important changes are grouped below:Script Usage Consistency and Clarity
prefix for consistency, and clarified instructions for collecting inventory from multiple subscriptions.
Multi-Region Support
docs/wiki/2-AvailabilityCheck.mddescribing how to generate availability reports for multiple regions, including instructions to create one output file per region and example commands.docs/wiki/3-CostInformation.mdto demonstrate how to handle multiple regions when running the cost comparison script.