GA Az.DataMigration #28598
GA Az.DataMigration #28598
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull Request Overview
Promotes Az.DataMigration module to GA and integrates it into the Az rollup module.
- Added Az.DataMigration to minimal version tracking and Az meta module dependency list.
- Updated ChangeLog with GA announcement and related feature notes.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tools/VersionController/MinimalVersion.csv | Registers Az.DataMigration with minimal version 1.0.0 for GA. |
| tools/Az/Az.psd1 | Adds Az.DataMigration to Az meta module dependencies (version inconsistency noted). |
| src/DataMigration/DataMigration/ChangeLog.md | Announces GA and documents related updates (tense/style issue). |
| @{ModuleName = 'Az.DataFactory'; RequiredVersion = '1.19.4'; }, | ||
| @{ModuleName = 'Az.DataLakeAnalytics'; RequiredVersion = '1.1.0'; }, | ||
| @{ModuleName = 'Az.DataLakeStore'; RequiredVersion = '1.5.0'; }, | ||
| @{ModuleName = 'Az.DataMigration'; RequiredVersion = '0.15.0'; }, |
There was a problem hiding this comment.
RequiredVersion is set to 0.15.0 while MinimalVersion.csv declares Az.DataMigration at 1.0.0 and the ChangeLog announces GA. This will cause the Az rollup to pull a pre-GA version. Update RequiredVersion to '1.0.0' to align with the GA promotion.
| @{ModuleName = 'Az.DataMigration'; RequiredVersion = '0.15.0'; }, | |
| @{ModuleName = 'Az.DataMigration'; RequiredVersion = '1.0.0'; }, |
Copilot uses AI. Check for mistakes.
| - Additional information about change #1 | ||
| --> | ||
| ## Upcoming Release | ||
| * General availability for module Az.DataMigration |
There was a problem hiding this comment.
ChangeLog entries under Upcoming Release should use past tense per repository guidelines. Suggested revision: "* Made module Az.DataMigration generally available." or "* Module Az.DataMigration reached general availability."
| * General availability for module Az.DataMigration | |
| * Made module Az.DataMigration generally available. |
Copilot uses AI. Check for mistakes.
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.