Skip to content

Conversation

fzkhan
Copy link
Member

@fzkhan fzkhan commented Oct 13, 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 Copilot AI review requested due to automatic review settings October 13, 2025 14:53
@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 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 Azure Data Transfer module to switch from API version 2025-05-21 to API version 2025-10-10. The update introduces new FlowProfile resource management capabilities and deprecates several existing properties in favor of newer FlowProfile-based approaches.

  • Introduces new FlowProfile resource with Create, Read, Update operations (Delete is hidden)
  • Adds FlowProfile-related parameters to existing Flow and Connection resources
  • Updates API version references and regenerates documentation

Reviewed Changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/DataTransfer/DataTransfer/Az.DataTransfer.psd1 Updates module to export new FlowProfile cmdlets and removes commented script/type processing lines
src/DataTransfer/DataTransfer.sln Changes project GUID reference for the AutoRest generated project
src/DataTransfer/DataTransfer/help/*.md Adds help documentation for new FlowProfile cmdlets and updates existing cmdlet descriptions with deprecation notices
src/DataTransfer/DataTransfer.Autorest/README.md Updates API commit reference and adds configuration for FlowProfile cmdlets including spelling fix directive
src/DataTransfer/DataTransfer.Autorest/UX/*.json Updates API versions to 2025-10-10-preview and adds new FlowProfile resource definitions
src/DataTransfer/DataTransfer.Autorest/test/*.ps1 Adds placeholder test files for new FlowProfile cmdlets
src/DataTransfer/DataTransfer.Autorest/examples/*.md Adds placeholder example files for new FlowProfile cmdlets
src/DataTransfer/DataTransfer.Autorest/docs/*.md Adds/updates documentation for cmdlets with new parameters and deprecation notices
src/DataTransfer/DataTransfer.Autorest/Properties/AssemblyInfo.cs Updates assembly version from 0.1.0 to 1.0.0
src/DataTransfer/DataTransfer.Autorest/generate-info.json Updates generation ID for the new API version

@fzkhan fzkhan changed the title Switch to API version 2025-10-10 [Az.DataTransfer] Switch to API version 2025-10-10 Oct 13, 2025
@fzkhan
Copy link
Member Author

fzkhan commented Oct 13, 2025

Notes:

@fzkhan fzkhan changed the title [Az.DataTransfer] Switch to API version 2025-10-10 [DoNotMerge][Az.DataTransfer] Switch to API version 2025-10-10 Oct 13, 2025
@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel isra-fel marked this pull request as draft October 13, 2025 23:33
@isra-fel isra-fel marked this pull request as ready for review October 13, 2025 23:55
- where:
parameter-name: AntiviruAvSolution
set:
parameter-name: AntivirusAvSolution
Copy link
Member

Choose a reason for hiding this comment

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

Just in case, is the name AntivirusAvSolution or AntivirusAvsSolution (double s)?

Copy link
Member Author

Choose a reason for hiding this comment

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

Single s: AntivirusAvSolution

[-ArchiveMaximumCompressionRatioLimit <Double>] [-ArchiveMaximumDepthLimit <Int64>]
[-ArchiveMaximumExpansionSizeLimit <Int64>] [-ArchiveMinimumSizeForExpansion <Int64>]
[-DataSizeMaximum <Int64>] [-DataSizeMinimum <Int64>] [-Description <String>] [-IdentityType <String>]
[-MimeFilter <IMimeTypeFilter[]>] [-MimeFilterType <String>] [-ReplicationScenario <String>]
Copy link
Member

Choose a reason for hiding this comment

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

Are there commands to help user create complex objects such as MimeFilter or TextMatchingDeny?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is good suggestion. I will consider adding them in a separate PR.

@fzkhan
Copy link
Member Author

fzkhan commented Oct 14, 2025

Closing this in favour of fzkhan#1

@fzkhan fzkhan closed this Oct 14, 2025
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