Skip to content

[PS] Migrate RedisEnterpriseCache module to autorest v4#28618

Open
JoyerJin wants to merge 7 commits intomainfrom
joyer/RedisEnterpriseCache-migrate-v4
Open

[PS] Migrate RedisEnterpriseCache module to autorest v4#28618
JoyerJin wants to merge 7 commits intomainfrom
joyer/RedisEnterpriseCache-migrate-v4

Conversation

@JoyerJin
Copy link
Contributor

@JoyerJin JoyerJin commented Sep 26, 2025

Description

Preannouncement PR: #28621

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

@azure-client-tools-bot-prd
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 28, 2025 06:07
Copilot AI review requested due to automatic review settings September 28, 2025 06:07
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 pull request migrates the RedisEnterpriseCache module from AutoRest v3 to AutoRest v4, involving significant changes to the code generation tooling and breaking changes to cmdlet interfaces and parameter types.

  • Updates the module to use AutoRest v4 with new API version 2025-05-01-preview
  • Changes numerous parameter types from strongly-typed enums to strings
  • Modifies parameter set structures and adds new parameter overloads for improved usability

Reviewed Changes

Copilot reviewed 80 out of 82 changed files in this pull request and generated 5 comments.

File Description
tools/StaticAnalysis/Exceptions/Az.RedisEnterpriseCache/BreakingChangeIssues.csv Documents breaking changes for cmdlet types and parameters to maintain compatibility tracking
src/RedisEnterpriseCache/RedisEnterpriseCache/help/*.md Updates help documentation to reflect new parameter types and syntax changes
src/RedisEnterpriseCache/RedisEnterpriseCache/Az.RedisEnterpriseCache.psd1 Updates module metadata including version requirements and build configuration
src/RedisEnterpriseCache/RedisEnterpriseCache.Autorest/test/*.ps1 Removes test implementations and updates test environment configuration

@isra-fel isra-fel added the autorest v4 migration pr migrating module from generated by autorest.powershell v3 to v4 label Oct 2, 2025
@Pan-Qi Pan-Qi marked this pull request as draft January 8, 2026 06:46
@Pan-Qi Pan-Qi added the Contains Breaking Change This PR contains breaking change label Jan 9, 2026
@Pan-Qi Pan-Qi marked this pull request as ready for review January 9, 2026 04:54
Copilot AI review requested due to automatic review settings January 9, 2026 04:54
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

To the author of the pull request,
This PR was labeled "Contains Breaking Change" because breaking changes have been detected by the static analysis pipeline.

  • According to our policy, breaking changes can only take place during major release and they must be preannounced.
  • Please follow our guide on the detailed steps.
  • Required: Please fill in the task below to facilitate our contact,you will receive notifications related to breaking changes.

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

Copilot reviewed 80 out of 83 changed files in this pull request and generated no new comments.

@koderjoker koderjoker self-assigned this Jan 9, 2026
@isra-fel isra-fel added this to the Az 16.0.0 (TBD) milestone Jan 28, 2026
@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

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 Contains Breaking Change This PR contains breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants