Azure / azure-powershell

Microsoft Azure PowerShell
Other
4.26k stars 3.87k forks source link

[Astro] Update astro custom logic for New-AzAstroOrganization and Update-AzAstroOrganization #26621

Closed lijinpei2008 closed 2 weeks ago

lijinpei2008 commented 2 weeks ago

Description

Mandatory Checklist

azure-client-tools-bot-prd[bot] commented 2 weeks ago
️✔️Az.Accounts >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>
⚠️Az.Astro >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>
>
> ️✔️Breaking Change Check > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>
>
> ⚠️Signature Check > >>
>> ⚠️PowerShell Core - Windows >> >>|Type|Cmdlet|Description|Remediation| >>|---|---|---|---| >>|⚠️|Get-AzAstroOrganization|Get-AzAstroOrganization Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzAstroOrganization|Get-AzAstroOrganization changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >> >>
>
>
> ️✔️Help File Existence Check > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>
>
> ⚠️File Change Check > >>
>> ⚠️PowerShell Core - Windows >> >>|Type|Cmdlet|Description|Remediation| >>|---|---|---|---| >>|⚠️||AssemblyInfo.cs will be updated automatically. Please do not update it manually.|Revert AssemblyInfo.cs to its last version.| >>|⚠️||It is required to update ChangeLog.md if you want to release a new version for Az.Astro.|Add a changelog record under Upcoming Release section with past tense.| >> >>
>
>
> ️✔️UX Metadata Check > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>
>
> ️✔️Test > >>
>> ️✔️PowerShell Core - Linux >> >> >>
>>
>> ️✔️PowerShell Core - MacOS >> >> >>
>>
>> ️✔️PowerShell Core - Windows >> >> >>
>
lijinpei2008 commented 2 weeks ago

/azp run

azure-pipelines[bot] commented 2 weeks ago
Azure Pipelines successfully started running 3 pipeline(s).
lijinpei2008 commented 2 weeks ago

/azp run

azure-pipelines[bot] commented 2 weeks ago
Azure Pipelines successfully started running 3 pipeline(s).