Skip to content

Use kimchi/README.md in the kimchi crate rustdoc #612

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

Closed
wants to merge 1 commit into from

Conversation

yorickvP
Copy link

As part of MinaProtocol/mina#11212, I'm moving some rust deps from a git submodule to a cargo git dep. However, the kimchi crate fails to build, since it can't find ../../README.md in this case.

I've moved the example into the kimchi crate README.md and used that.

Using README.md causes this crate to fail to build when specified
as an external dependency, since it can't find ../../README.md.
@mimoo
Copy link
Contributor

mimoo commented Jun 12, 2022

#555 should fix your issue I believe. Also, we should really create releases and publish them on crates.io. I think we'll investigate that next.

@yorickvP
Copy link
Author

#555 will fix the kimchi crate, but break the circuit-construction crate in exactly the same way. Publishing it on crates.io would be ideal, but it would also be nice to have a fix for the interim.

@mimoo
Copy link
Contributor

mimoo commented Jun 13, 2022

you don't need to import the circuit-construction crate in mina so that should be fine right for now right? Let me know, I'm hopping we find a solution where we can keep examples in the main README

@yorickvP
Copy link
Author

This works, thanks!

@yorickvP yorickvP closed this Jun 14, 2022
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.

2 participants