Skip to content

Conversation

JoyerJin
Copy link
Contributor

@JoyerJin JoyerJin commented Aug 8, 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

Copy link

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

@JoyerJin JoyerJin marked this pull request as ready for review September 1, 2025 08:35
@Copilot Copilot AI review requested due to automatic review settings September 1, 2025 08:35
Copy link
Contributor

@Copilot 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 migrates the StackHCI module from autorest v3 to autorest v4, updating generated PowerShell cmdlets and their documentation. The migration brings several structural changes to the module's API surface.

Key changes include:

  • Updated type definitions from strongly-typed enums to string types for better flexibility
  • Modified parameter sets and method signatures across cmdlets
  • Added new JSON input parameter support for several cmdlets
  • Restructured identity and authentication parameter handling

Reviewed Changes

Copilot reviewed 83 out of 84 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
help/*.md Updated cmdlet documentation with new parameter types and syntax changes
StackHCI.sln Updated project GUID references for autorest v4 compatibility
Az.StackHCI.psd1 Added new cmdlet exports and updated module metadata
test/utils.ps1 Fixed variable references in test utility functions
docs/*.md Updated autorest documentation with new type definitions

@VeryEarly
Copy link
Collaborator

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@JoyerJin
Copy link
Contributor Author

JoyerJin commented Sep 28, 2025

waiting the service team for recording test

@isra-fel isra-fel added the autorest v4 migration pr migrating module from generated by autorest.powershell v3 to v4 label Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autorest v4 migration pr migrating module from generated by autorest.powershell v3 to v4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants