Skip to content

Conversation

@Pan-Qi
Copy link
Contributor

@Pan-Qi Pan-Qi commented Dec 18, 2025

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings December 18, 2025 07:40
@azure-client-tools-bot-prd
Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the NetworkCloud PowerShell module to use API version 2025-09-01 (from 2025-02-01). The changes include regenerated help documentation, two new cmdlets, and various parameter additions to existing cmdlets to support new API features.

Key Changes

  • API version upgrade: Updated from 2025-02-01 to 2025-09-01 with new commit hash reference
  • New cmdlets added: Start-AzNetworkCloudStorageApplianceReadCommand and Invoke-AzNetworkCloudBareMetalMachineDataExtractRestricted
  • Enhanced parameters: Added identity management, pagination, and storage options to various cmdlets

Reviewed changes

Copilot reviewed 100 out of 102 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/NetworkCloud/NetworkCloud.Autorest/README.md Updated API version and commit hash reference
src/NetworkCloud/NetworkCloud.Autorest/UX/Microsoft.NetworkCloud/*.json Updated apiVersion from 2025-02-01 to 2025-09-01 across all resource types
src/NetworkCloud/NetworkCloud.Autorest/Properties/AssemblyInfo.cs Bumped assembly version from 1.2.0 to 2.0.0
src/NetworkCloud/NetworkCloud/Az.NetworkCloud.psd1 Added new cmdlets to exports and updated generation date
src/NetworkCloud/NetworkCloud/help/*.md Regenerated help documentation with new parameters and cmdlets
src/NetworkCloud/NetworkCloud.Autorest/examples/*.md Added placeholder examples for new cmdlets
src/NetworkCloud/NetworkCloud.Autorest/test/*.Tests.ps1 Added test scaffolding for new cmdlets
src/NetworkCloud/NetworkCloud.Autorest/docs/*.md Updated documentation with new parameters and cmdlet descriptions
src/NetworkCloud/NetworkCloud.sln Updated project GUID reference

Comment on lines +1 to +22
### Example 1: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}

### Example 2: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}

Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The example file contains placeholder content that needs to be replaced with actual, working examples. Placeholder text like "Add title here", "Add code here", and "Add description here" should be replaced with real examples demonstrating how to use the Start-AzNetworkCloudStorageApplianceReadCommand cmdlet.

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +22
### Example 1: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}

### Example 2: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}

Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The example file contains placeholder content that needs to be replaced with actual, working examples. Placeholder text like "Add title here", "Add code here", and "Add description here" should be replaced with real examples demonstrating how to use the Invoke-AzNetworkCloudBareMetalMachineDataExtractRestricted cmdlet.

Copilot uses AI. Check for mistakes.
Comment on lines +49 to +68
### Example 1: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}

### Example 2: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The help documentation contains placeholder examples that need to be replaced with actual, working examples. The placeholder text "Add title here", "Add code here", and "Add description here" should be replaced with meaningful examples demonstrating how to use this cmdlet.

Copilot uses AI. Check for mistakes.
Comment on lines +51 to +71
### Example 1: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}

### Example 2: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here (remove the output block if the example doesn't have an output) }}
```

{{ Add description here }}
Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The help documentation contains placeholder examples that need to be replaced with actual, working examples. The placeholder text "Add title here", "Add code here", and "Add description here" should be replaced with meaningful examples demonstrating how to use this cmdlet.

Copilot uses AI. Check for mistakes.
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