Skip to content

Conversation

@lvlindv
Copy link
Collaborator

@lvlindv lvlindv commented Dec 10, 2025

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

  • Updated all PowerShell script examples in docs/wiki/1-Collect.md to 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

  • Added a new section in docs/wiki/2-AvailabilityCheck.md describing how to generate availability reports for multiple regions, including instructions to create one output file per region and example commands.
  • Updated the example in docs/wiki/3-CostInformation.md to demonstrate how to handle multiple regions when running the cost comparison script.

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.
Copy link
Contributor

Copilot AI left a 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.json as 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.

lvlindv and others added 6 commits December 15, 2025 09:51
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>
@lvlindv lvlindv requested a review from jfaurskov December 15, 2025 08:56
Copy link
Collaborator

@jfaurskov jfaurskov left a comment

Choose a reason for hiding this comment

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

LGTM

@jfaurskov jfaurskov merged commit 2308400 into main Dec 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants