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

Fix build issue where Debug not implemented on syn::Type. #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

onelson
Copy link

@onelson onelson commented Dec 2, 2019

Build fails since syn::Type only adds an impl for Debug when
"extra-traits" feature is enabled. This exposes a feature on
typescript-definitions-derive to control whether or not syn's feature is
active, and adds a cfg_attr to the derive for Attrs.

This was mentioned incidentally in #4 but does not solve the breakage starting in nightly-2019-11-25 which seems to be related to the pest usage.

Build fails since syn::Type only adds an impl for Debug when
"extra-traits" feature is enabled. This exposes a feature on
typescript-definitions-derive to control whether or not syn's feature is
active, and adds a cfg_attr to the derive on `Attrs`.

- <dtolnay/syn#583 (comment)>
@ufoscout
Copy link

ufoscout commented Dec 8, 2019

Any idea when this will be merged and released?

@onelson
Copy link
Author

onelson commented Dec 9, 2019

@ufoscout maintainer may not be in a rush to merge anything. There's an open PR from April which doesn't seem to have gotten any attention.

I'm not planning on publishing my fork to crates.io, but I will surely leave it up on github for as long as the problem is unaddressed (should you want to depend on it directly).

@ufoscout
Copy link

@onelson
thanks, I am using your branch now

@ytitov
Copy link

ytitov commented Mar 19, 2020

would be really nice if this was merged, using @ufoscout branch. Really hoping this doesn't die. Using rust on a web api backend and having to manually write the definitions.

@Fishrock123
Copy link

@arabidopsis please merge and release this

@benma
Copy link

benma commented Jul 5, 2023

Bump. Is this crate abandoned?

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.

5 participants