-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor script for multiple validators #11
base: main
Are you sure you want to change the base?
Conversation
…d from that constant
…ch file, generated list from files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some typos
Fixed types with any on transform functions and used a newer .hasOwnProperty API
…th an empty object fallback
Co-authored-by: Jacob M-G Evans <[email protected]>
Fixed types and added a test for main loadUsers function
Additional Tests
Is this still pending approval by @domitriusclark or someone else? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to add more tests, but the tests for various inputs and how they map to Clerk are a good start.
Lets merge this. |
Not yet. There is some outstanding work outlined in the project. |
No description provided.