Skip to content

Migrates 'entra resourcenamespace list' to Zod. Closes #6907#7185

Merged
milanholemans merged 3 commits intopnp:mainfrom
MartinM85:feature/6907-entra-resourcenamespace-list-zod
Apr 24, 2026
Merged

Migrates 'entra resourcenamespace list' to Zod. Closes #6907#7185
milanholemans merged 3 commits intopnp:mainfrom
MartinM85:feature/6907-entra-resourcenamespace-list-zod

Conversation

@MartinM85
Copy link
Copy Markdown
Contributor

Closes #6907

@milanholemans milanholemans self-assigned this Apr 17, 2026
Comment thread src/m365/entra/commands/resourcenamespace/resourcenamespace-list.spec.ts Outdated
Comment thread src/m365/entra/commands/resourcenamespace/resourcenamespace-list.spec.ts Outdated
@milanholemans milanholemans marked this pull request as draft April 17, 2026 22:18
@MartinM85 MartinM85 marked this pull request as ready for review April 19, 2026 09:46
@MartinM85 MartinM85 requested a review from milanholemans April 19, 2026 09:46
@milanholemans milanholemans requested a review from Copilot April 24, 2026 22:00
Copy link
Copy Markdown

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

Migrates the entra resourcenamespace list command to Zod-based option validation, aligning it with the CLI’s ongoing schema-driven validation refactor (issue #6907).

Changes:

  • Adds a Zod options schema (based on globalOptionsZod) and exposes it via the command’s schema getter.
  • Updates the command’s test suite to validate schema parsing (success with no options; failure with unknown options).

Reviewed changes

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

File Description
src/m365/entra/commands/resourcenamespace/resourcenamespace-list.ts Adds Zod schema for global options and wires it into the command via schema.
src/m365/entra/commands/resourcenamespace/resourcenamespace-list.spec.ts Updates tests to use the command’s parsed schema and adds validation test cases.

Comment thread src/m365/entra/commands/resourcenamespace/resourcenamespace-list.spec.ts Outdated
@milanholemans milanholemans merged commit 7f3264b into pnp:main Apr 24, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate 'entra resourcenamespace list' to Zod

3 participants