Conversation
️✔️Az.Accounts
|
| Type | Cmdlet | Description | Remediation |
|---|---|---|---|
| Get-AzStackHciArcSetting | Get-AzStackHciArcSetting 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-AzStackHciArcSetting | Get-AzStackHciArcSetting 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. | |
| Get-AzStackHciCluster | Get-AzStackHciCluster 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-AzStackHciCluster | Get-AzStackHciCluster 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. | |
| Get-AzStackHciExtension | Get-AzStackHciExtension 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-AzStackHciExtension | Get-AzStackHciExtension 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. | |
| Get-AzStackHCIRemoteSupportAccess | Get-AzStackHCIRemoteSupportAccess 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-AzStackHCIRemoteSupportAccess | Get-AzStackHCIRemoteSupportAccess 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. | |
| Get-AzStackHCIRemoteSupportSessionHistory | Get-AzStackHCIRemoteSupportSessionHistory 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-AzStackHCIRemoteSupportSessionHistory | Get-AzStackHCIRemoteSupportSessionHistory 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. | |
| Get-AzStackHCIVmImage | Get-AzStackHCIVmImage 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-AzStackHCIVmImage | Get-AzStackHCIVmImage 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. | |
| Get-AzStackHCIVmLogicalNetwork | Get-AzStackHCIVmLogicalNetwork 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-AzStackHCIVmLogicalNetwork | Get-AzStackHCIVmLogicalNetwork 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. | |
| Get-AzStackHCIVmNetworkInterface | Get-AzStackHCIVmNetworkInterface 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-AzStackHCIVmNetworkInterface | Get-AzStackHCIVmNetworkInterface 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. | |
| Get-AzStackHCIVmStoragePath | Get-AzStackHCIVmStoragePath 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-AzStackHCIVmStoragePath | Get-AzStackHCIVmStoragePath 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. | |
| Get-AzStackHCIVmVirtualHardDisk | Get-AzStackHCIVmVirtualHardDisk 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-AzStackHCIVmVirtualHardDisk | Get-AzStackHCIVmVirtualHardDisk 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 Example Check
️✔️PowerShell Core - Windows
️✔️Help File Existence Check
️✔️PowerShell Core - Windows
⚠️ File Change Check
⚠️ PowerShell Core - Windows
Type Cmdlet Description Remediation ⚠️ It is required to update ChangeLog.md if you want to release a new version for Az.StackHCI. Add a changelog record under Upcoming Release section with past tense.
️✔️UX Metadata Check
️✔️PowerShell Core - Windows
⚠️ Test
⚠️ - Linux
Type Title Current Coverage Description ⚠️ Test Coverage Less Than 50% 18.97 % Test coverage for the module cannot be lower than 50%.
⚠️ - MacOS
Type Title Current Coverage Description ⚠️ Test Coverage Less Than 50% 18.97% Test coverage for the module cannot be lower than 50%.
⚠️ PowerShell Core - Windows
Type Title Current Coverage Description ⚠️ Test Coverage Less Than 50% 18.97% Test coverage for the module cannot be lower than 50%.
|
@VeryEarly Please let me know if this PR is good to merge. |
VeryEarly
left a comment
There was a problem hiding this comment.
there is a folder export under docs, please remove this folder and move help markdown under docs
VeryEarly
left a comment
There was a problem hiding this comment.
folder structure is incorrect, there should be folders for both StackHCIVm and StackHCI
VeryEarly
left a comment
There was a problem hiding this comment.
inconsistent term "StackHCIVM" "StackHCIVm"
| title: StackHCIVm | ||
| service-name: StackHCIVm | ||
| subject-prefix: $(service-name) | ||
|
|
There was a problem hiding this comment.
add line: root-module-name: $(prefix).StackHCI
VeryEarly
left a comment
There was a problem hiding this comment.
why deleting src/StackHCI/examples/Get-AzStackHCILogsDirectory.md?
|
this PR is far from ready, I don't think it catch 2023-11-14 release, please consider move to some latter iterations. |
@VeryEarly I was told that I need to only include the files for the new StackHCIVm module in the PR. I have had this conversation with other engineers in previous iterations! |
I have moved these markdown files just under the doc section. |
|
PS E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest> .\test-module.ps1 -Playback Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Add-AzStackHCIVmVirtualMachineDataDisk.Tests.ps1 Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Add-AzStackHCIVmVirtualMachineNetworkInterface.Tests.ps1 Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\AzStackHCIVmImageNew.Tests.ps1 Describing AzStackHCIVmImageNew Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\AzStackHciVmLogicalNetwork.Tests.ps1 Describing AzStackHCIVmLogicalNetwork Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\AzStackHciVmNetworkInterface.Tests.ps1 Describing AzStackHCIVmNetworkInterface Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\AzStackHciVmStoragePath.Tests.ps1 Describing AzStackHCIVmStoragePath Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\AzStackHCIVmVirtualHardDisks.Tests.ps1 Describing AzStackHCIVmVirtualHardDisks Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\AzStackHciVmVirtualMachine.Tests.ps1 Describing AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Get-AzStackHCIVmImage.Tests.ps1 Describing Get-AzStackHCIVmImage Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Get-AzStackHciVMLogicalNetwork.Tests.ps1 Describing Get-AzStackHciVMLogicalNetwork Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Get-AzStackHciVMNetworkInterface.Tests.ps1 Describing Get-AzStackHciVMNetworkInterface Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Get-AzStackHciVMStoragePath.Tests.ps1 Describing Get-AzStackHciVMStoragePath Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Get-AzStackHciVMVirtualHardDisk.Tests.ps1 Describing Get-AzStackHciVMVirtualHardDisk Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Get-AzStackHCIVmVirtualMachine.Tests.ps1 Describing Get-AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\New-AzStackHCIVmImage.Tests.ps1 Describing New-AzStackHCIVmImage Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\New-AzStackHCIVmLogicalNetwork.Tests.ps1 Describing New-AzStackHCIVmLogicalNetwork Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\New-AzStackHCIVmNetworkInterface.Tests.ps1 Describing New-AzStackHCIVmNetworkInterface Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\New-AzStackHCIVmStoragePath.Tests.ps1 Describing New-AzStackHCIVmStoragePath Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\New-AzStackHCIVmVirtualHardDisk.Tests.ps1 Describing New-AzStackHCIVmVirtualHardDisk Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\New-AzStackHCIVmVirtualMachine.Tests.ps1 Describing New-AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmImage.Tests.ps1 Describing Remove-AzStackHCIVmImage Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmLogicalNetwork.Tests.ps1 Describing Remove-AzStackHCIVmLogicalNetwork Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmNetworkInterface.Tests.ps1 Describing Remove-AzStackHCIVmNetworkInterface Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmStoragePath.Tests.ps1 Describing Remove-AzStackHCIVmStoragePath Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmVirtualHardDisk.Tests.ps1 Describing Remove-AzStackHCIVmVirtualHardDisk Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmVirtualMachine.Tests.ps1 Describing Remove-AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmVirtualMachineDataDisk.Tests.ps1 Describing Remove-AzStackHCIVmVirtualMachineDataDisk Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Remove-AzStackHCIVmVirtualMachineNetworkInterface.Tests.ps1 Describing Remove-AzStackHCIVmVirtualMachineNetworkInterface Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Restart-AzStackHCIVmVirtualMachine.Tests.ps1 Describing Restart-AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Start-AzStackHCIVmVirtualMachine.Tests.ps1 Describing Start-AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Stop-AzStackHCIVmVirtualMachine.Tests.ps1 Describing Stop-AzStackHCIVmVirtualMachine Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Update-AzStackHCIVmImage.Tests.ps1 Describing Update-AzStackHCIVmImage Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Update-AzStackHciVMLogicalNetwork.Tests.ps1 Describing Update-AzStackHciVMLogicalNetwork Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Update-AzStackHciVMNetworkInterface.Tests.ps1 Describing Update-AzStackHciVMNetworkInterface Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Update-AzStackHciVMStoragePath.Tests.ps1 Describing Update-AzStackHciVMStoragePath Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Update-AzStackHciVMVirtualHardDisk.Tests.ps1 Describing Update-AzStackHciVMVirtualHardDisk Executing script E:\new\azure-powershell2\src\StackHCI\StackHCIVm.Autorest\test\Update-AzStackHCIVmVirtualMachine.Tests.ps1 Describing Update-AzStackHCIVmVirtualMachine |
64da914 to
ce125bf
Compare
Description
Checklist
CONTRIBUTING.mdand reviewed the following information:generationbranch.ChangeLog.mdfile(s) appropriatelyChangeLog.mdfile can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md## Upcoming Releaseheader in the past tense. Add changelog in description section if PR goes intogenerationbranch.ChangeLog.mdif no new release is required, such as fixing test case only.