Breaking Change - Removed ApiVersion parameter from Az.Resources deployments cmdlets and changed the output type of Get-AzResourceGroupDeploymentOperations

[Xynoclafe]

Description

Removed ApiVersion parameter from Az.Resources deployments cmdlets and changed the output type of Get-AzResourceGroupDeploymentOperations

This is a breaking change.
Design review - https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/707

Checklist

• I have read the Submitting Changes section of CONTRIBUTING.md
• The title of the PR is clear and informative
• The appropriate ChangeLog.md file(s) has been updated:
  • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
  • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
• The PR does not introduce breaking changes
• If applicable, the changes made in the PR have proper test coverage
• For public API changes to cmdlets:
  • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
  • the markdown help files have been regenerated using the commands listed here

[filizt]

public abstract class ResourceManagerCmdletBaseWithAPiVersion : ResourceManagerCmdletBase

Could you please fix the casing of "APi" in the class name?

---

Refers to: src/Resources/ResourceManager/Implementation/CmdletBase/ResourceManagerCmdletBaseWithAPiVersion.cs:21 in 6c9887e. [](commit_id = 6c9887e, deletion_comment = False)

[filizt]

I think it's a worthy effort to make the change longs a bit verbose to make things clear. We can add something like "Remove ApiVersion parameter from Deployment cmdlets" (if the cmdlet list is not too long, maybe add them too?)

[Xynoclafe]

There are quite a few cmdlets (mentioned in the design review), so I don't think adding them to the changelog is practical. But I'll make it a little more verbose.

[filizt]

Could you also add the link to the design discussion in the description section?

[Xynoclafe]

@isra-fel Can you please review this PR?

[isra-fel]

/azp run azure-powershell - windows-powershell

[azure-pipelines]

No pipelines are associated with this pull request.

[isra-fel]

Sorry @Xynoclafe our team are just back from holiday. Anyway your changes LGTM. Thanks

[isra-fel]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[Xynoclafe]

Hi @isra-fel
Can you review the PR again? Your last commit voided your previous review. And I think we can remove the Waiting for CI label as CI seems to be passing now.

[isra-fel]

@Xynoclafe I'm sorry but it seems the merge of #13090 has caused a lot of conflict in the test recording file. Would you please resolve them by merging master and record ResourceGroupDeploymentEndToEnd again?

Thanks in advance!