Skip to content
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

Override gen annotation for DeriveGen functionality #458

Closed
wants to merge 2 commits into from

Conversation

beem812
Copy link
Contributor

@beem812 beem812 commented Dec 8, 2022

@generator() annotation for specifying a different generator for the DeriveGen functionality to use. #438

Allows for overriding the generator at both the field schema and the value schema level

james-beem-axoni and others added 2 commits December 8, 2022 17:29
Potentially working needs cleanup

Add test to show it ignores a generator of a different type

Remove type safety and test

Formatting and test name fix

Formatting and linting fixes

formatting

Formatting again
@beem812 beem812 requested a review from a team as a code owner December 8, 2022 22:50
@jdegoes
Copy link
Member

jdegoes commented Aug 30, 2023

@beem812 I feel like the best way to support this would be to re-implement DeriveGen in terms of Deriver, which already allows implicits to override the default instances. What do you think?

@jdegoes jdegoes closed this Aug 30, 2023
@beem812
Copy link
Contributor Author

beem812 commented Aug 31, 2023

Sure, I'd be happy to take a look at it and attack it from that path. The lack of type safety from this solution felt weird anyway, hopefully the deriver route eliminates that issue.

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