Skip to content

Migrates 'status' command to Zod. Closes #6912#6930

Merged
martinlingstuyl merged 1 commit intopnp:mainfrom
Malay-dev:main
Oct 1, 2025
Merged

Migrates 'status' command to Zod. Closes #6912#6930
martinlingstuyl merged 1 commit intopnp:mainfrom
Malay-dev:main

Conversation

@Malay-dev
Copy link
Copy Markdown
Contributor

Migrates 'status' command to use Zod for data validation. Closes #6912

Test Report

image

@milanholemans
Copy link
Copy Markdown
Contributor

Thanks, we'll try to review it ASAP!

@martinlingstuyl martinlingstuyl self-assigned this Oct 1, 2025
Copy link
Copy Markdown
Contributor

@martinlingstuyl martinlingstuyl left a comment

Choose a reason for hiding this comment

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

One small comment, other than that this PR is fine. Would you fix the comment and respond?

Comment thread src/m365/commands/status.ts Outdated
@Malay-dev Malay-dev marked this pull request as ready for review October 1, 2025 14:44
Copy link
Copy Markdown
Contributor

@martinlingstuyl martinlingstuyl left a comment

Choose a reason for hiding this comment

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

Nice work! Approved.... I'll merge it later

@martinlingstuyl martinlingstuyl merged commit b50ebf7 into pnp:main Oct 1, 2025
11 of 13 checks passed
@martinlingstuyl martinlingstuyl added the hacktoberfest-accepted Accept for hacktoberfest, will merge later label Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted Accept for hacktoberfest, will merge later pr-merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate 'status' to Zod

3 participants