Skip to content
This repository was archived by the owner on Oct 29, 2025. It is now read-only.

Azure migrate script - #33

Merged
Jan Faurskov (jfaurskov) merged 8 commits into
Azure:mainfrom
imnotfromhere:Azure-Migrate-Script
Jul 4, 2025
Merged

Azure migrate script#33
Jan Faurskov (jfaurskov) merged 8 commits into
Azure:mainfrom
imnotfromhere:Azure-Migrate-Script

Conversation

@imnotfromhere

Copy link
Copy Markdown
Contributor

Added script to convert the output Excel file from a Azure Migrate Assessment to the same format as Get-AzureServices.ps1
You can use the attached file to test the functionality

Assessment01.xlsx

Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated

@jfaurskov Jan Faurskov (jfaurskov) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work imnotfromhere, just a few comments

Comment thread 1-Collect/Get-RessourcesFromAM.ps1 Outdated
Write-Output "File not found: $filePath"
exit
}
# Ensure the ImportExcel module is installed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

- Requires ImportExcel module to be installed for reading Excel files.

is already in line 21. Is that enough?

@jfaurskov Jan Faurskov (jfaurskov) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jfaurskov
Jan Faurskov (jfaurskov) merged commit 813e4b0 into Azure:main Jul 4, 2025
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants