Skip to content

Conversation

@jordicasanella
Copy link
Contributor

@jordicasanella jordicasanella commented Nov 11, 2024

Summary

changes to enable to validate enum values

How Has This Been Tested?

Unit Test
Install package locally and build fix translator

Screenshot fix-translator checking enum:

image

Screenshot from unit Test:

image

Copy link

@aditya-valstro aditya-valstro left a comment

Choose a reason for hiding this comment

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

Just had one small NIT but looks great. I would've liked to see some screenshots in the PR summary that show the logs that the enum recursive check generates. Overall though, nice changes.

@jordicasanella jordicasanella merged commit 1b6f2d2 into main Nov 12, 2024
1 check passed
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