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

chore: loosen feature requirement on deser methods #265

Closed
wants to merge 1 commit into from

Conversation

gakonst
Copy link
Contributor

@gakonst gakonst commented Mar 1, 2022

the full-serde feature is not really necessary here, you can deserialize just with serde enabled and you don't gain anything by feature gating reader behind full-serde

@vkgnosis
Copy link
Member

vkgnosis commented Mar 1, 2022

Should this be included in the release I'm making at the moment?

@gakonst
Copy link
Contributor Author

gakonst commented Mar 1, 2022

NVM this unfortunately doesn't seem to work. Both of these depend on std features, encountered while trying to trim dependencies from parity-common. The 2 PRs which should "properly" address the issue are: paritytech/parity-common#630 and paritytech/parity-common#625.

@gakonst gakonst closed this Mar 1, 2022
@gakonst gakonst deleted the chore/feature-trim branch March 1, 2022 10:53
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

2 participants