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

Unignore doctests where possible #402

Closed
wants to merge 1 commit into from

Conversation

rukai
Copy link

@rukai rukai commented Aug 6, 2021

closes #363

Possibly we would be better off just deleting the examples from Serializer and Deserializer?
I dont mind either way.

I left the zigzag example as ignore because its a pseudocode demonstration and getting it compiling would obscure the intent.

@stale
Copy link

stale bot commented Oct 5, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 5, 2021
@rukai
Copy link
Author

rukai commented Oct 5, 2021

alright, I rebased it.

@ZoeyR
Copy link
Collaborator

ZoeyR commented Oct 7, 2021

Sorry I missed this, I'll mark it as not stale for now

@VictorKoenders
Copy link
Contributor

Because we're in the middle of a big rewrite for bincode 2, I've taken the liberty to implement your change in a different PR: #421

Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Example for Deserializer uses bytes_read method which doesn't exist
3 participants