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

Remove strum_macros and use full #[derive(Debug)] #812

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adetaylor
Copy link
Collaborator

Simpler but the debug output is incomprehensibly huge. We need to wrap all syn things in a newtype wrapper which outputs more concise debug output.

@adetaylor
Copy link
Collaborator Author

I also explored the option of a newtype wrapper in this branch but it has hundreds of compile errors and would require radical changes throughout the code.

It's possible that just using this minisyn within the Api type might make sense though.

@adetaylor
Copy link
Collaborator Author

And a third attempt here, using minisyn just for api.rs. This builds and runs, but output is still too verbose.

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.

1 participant