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

impl From<syn::Error> for Error #116

Merged
merged 1 commit into from Jan 5, 2021
Merged

impl From<syn::Error> for Error #116

merged 1 commit into from Jan 5, 2021

Conversation

TedDriggs
Copy link
Owner

syn provides useful functions for low-level AST parsing that go beyond
what darling attempts to support natively. In these cases, a caller
may want to propagate a syn::Error up through a function that returns
a darling::Result.

This new impl guarantees lossless handling, emitting the same compiler
error is emitted as if the syn::Error::to_compile_error had been
called directly.

`syn` provides useful functions for low-level AST parsing that go beyond
what `darling` attempts to support natively. In these cases, a caller
may want to propagate a `syn::Error` up through a function that returns
a `darling::Result`.

This new impl guarantees lossless handling, emitting the same compiler
error is emitted as if the `syn::Error::to_compile_error` had been
called directly.
@TedDriggs
Copy link
Owner Author

Merging before CI completion after local testing against 1.31.0 and stable without issue. Filed #117 to address the unacceptable slowness of Travis-CI.

@TedDriggs TedDriggs merged commit b172383 into master Jan 5, 2021
@TedDriggs TedDriggs deleted the from-syn-error branch January 5, 2021 15:42
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.

None yet

1 participant