Skip to content

Check if modules are already loaded before importing#57

Closed
Copilot wants to merge 2 commits intomasterfrom
copilot/review-recent-pr
Closed

Check if modules are already loaded before importing#57
Copilot wants to merge 2 commits intomasterfrom
copilot/review-recent-pr

Conversation

Copy link

Copilot AI commented Mar 15, 2026

MicroBurst.psm1 used Get-InstalledModule to gate module imports, which fails for manually-loaded modules (not installed via PowerShellGet), causing MicroBurst functions to silently not load.

Changes

  • MicroBurst.psm1: Wrap each module's Get-InstalledModule + Import-Module block in a Get-Module check for Az, AzureAD, AzureRM (commented section), and msonline
# Before: skips if not installed via PowerShellGet, even if already loaded
Get-InstalledModule -ErrorAction Stop -Name Az | Out-Null
Import-Module Az -ErrorAction Stop

# After: bypass install check if module is already in session
if (-not (Get-Module -Name Az)) {
    Get-InstalledModule -ErrorAction Stop -Name Az | Out-Null
    Import-Module Az -ErrorAction Stop
}

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: kfosaaen <2163397+kfosaaen@users.noreply.github.com>
Copilot AI changed the title [WIP] Review latest pull request changes Check if modules are already loaded before importing Mar 15, 2026
Copilot AI requested a review from kfosaaen March 15, 2026 18:56
@kfosaaen kfosaaen closed this Mar 15, 2026
@kfosaaen kfosaaen deleted the copilot/review-recent-pr branch March 15, 2026 18:58
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.

2 participants