Skip to content

Conversation

@Siongithub
Copy link

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

@azure-client-tools-bot-prd
Copy link

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

Copilot finished reviewing on behalf of Siongithub December 3, 2025 19:06
@isra-fel
Copy link
Member

isra-fel commented Dec 3, 2025

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

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 introduces a public preview release (API version 2025-11-01-preview) for the Az.DeviceRegistry module, adding support for two new resource types: Namespace Credentials and Namespace Policies. The changes include new cmdlets for managing these resources (Create, Get, Update, Remove, and Sync operations), comprehensive help documentation, test coverage, and necessary configuration updates.

Key changes include:

  • Addition of 10 new cmdlets for Credentials and Policy management
  • Comprehensive markdown help documentation with detailed examples
  • Test files and test configuration for the new functionality
  • Module manifest updates to export new cmdlets and update dependencies

Reviewed changes

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

Show a summary per file
File Description
src/DeviceRegistry/DeviceRegistry/ChangeLog.md Added release notes for new Credentials and Policies resources with cmdlet listings
src/DeviceRegistry/DeviceRegistry/help/*.md Added 7 new help documentation files for Credentials and Policy cmdlets with examples
src/DeviceRegistry/DeviceRegistry/help/Update-AzDeviceRegistryNamespaceDevice.md Updated to add new PolicyResourceId parameter
src/DeviceRegistry/DeviceRegistry/help/Az.DeviceRegistry.md Updated module index to include new cmdlets
src/DeviceRegistry/DeviceRegistry/Az.DeviceRegistry.psd1 Updated exported functions, dependencies, and metadata
src/DeviceRegistry/DeviceRegistry.Autorest/test/*.Tests.ps1 Added 5 new test files for Credentials and Policy cmdlets
src/DeviceRegistry/DeviceRegistry.Autorest/test/*.Recording.json Updated API version references and added test recordings
src/DeviceRegistry/DeviceRegistry.Autorest/test/utils.ps1 Added test configuration for Credentials and Policy tests
src/DeviceRegistry/DeviceRegistry.Autorest/test/jsonFiles/*.json Added JSON test files for Policy operations
src/DeviceRegistry/DeviceRegistry.Autorest/test/env.json Updated test environment configuration
src/DeviceRegistry/DeviceRegistry.sln Updated project GUID references
src/DeviceRegistry/DeviceRegistry.Autorest/generate-info.json Updated generation ID
src/DeviceRegistry/DeviceRegistry.Autorest/UX/Microsoft.DeviceRegistry/assets.json Updated API version reference

@Siongithub
Copy link
Author

/azp run

@azure-pipelines
Copy link
Contributor

Commenter does not have sufficient privileges for PR 28950 in repo Azure/azure-powershell

@wyunchi-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@wyunchi-ms
Copy link
Contributor

Please fix the issues that static analysis found. @Siongithub

Copilot AI review requested due to automatic review settings December 5, 2025 14:48
@Siongithub
Copy link
Author

@wyunchi-ms fixed the examples and regenerated build

Copilot finished reviewing on behalf of Siongithub December 5, 2025 14:50
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 65 out of 116 changed files in this pull request and generated no new comments.

@isra-fel
Copy link
Member

isra-fel commented Dec 5, 2025

/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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants