Skip to content

Revert "Fixes to address Zod & property renaming. (#872)"#889

Open
jgindin wants to merge 2 commits intogoogle:mainfrom
jgindin:revert-property-renaming
Open

Revert "Fixes to address Zod & property renaming. (#872)"#889
jgindin wants to merge 2 commits intogoogle:mainfrom
jgindin:revert-property-renaming

Conversation

@jgindin
Copy link
Collaborator

@jgindin jgindin commented Mar 18, 2026

Description

The necessary changes to handle proper property renaming will take place internally.

Pre-launch Checklist

  • [ x] I signed the CLA.
  • [ x] I read the Contributors Guide.
  • [x ] I read the Style Guide.
  • [x ] I have added updates to the CHANGELOG.
  • [ x] I updated/added relevant documentation.
  • [x ] My code changes (if any) have tests.

The necessary changes to handle proper property renaming will take
place internally.
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request primarily reverts changes made in a previous pull request (#872) concerning Zod schema definitions. The changes revert declare interface definitions back to simpler type aliases across multiple TypeScript files, which simplifies the codebase. Additionally, some unused imports are removed, and the version is correctly updated in package.json with a corresponding CHANGELOG.md entry. The changes are consistent with the goal of reverting the previous modifications, and I found no issues with the implementation.

@jgindin jgindin changed the title Mostly revert "Fixes to address Zod & property renaming. (#872)" Revert "Fixes to address Zod & property renaming. (#872)" Mar 18, 2026
@jgindin jgindin requested a review from wrenj March 18, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants