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

Improve the docs when running cargo docs and fix the Clippy warnings for mdast_util_to_markdown #147

Closed
wants to merge 3 commits into from

Conversation

bnchi
Copy link
Contributor

@bnchi bnchi commented Oct 7, 2024

This PR is a follow up for #127 to comply with how the docs are written in this project and to fix some of the warnings raised by Clippy when running cargo clippy --all-features --all-targets --workspace

@bnchi
Copy link
Contributor Author

bnchi commented Oct 7, 2024

Screenshot 2024-10-07 at 16-09-45 Options in mdast_util_to_markdown - Rust

I believe this is the correct way to format the docs ?!

@wooorm
Copy link
Owner

wooorm commented Oct 7, 2024

Thanks!

I am currently locally looking through the code and refactoring such things! :)

@bnchi
Copy link
Contributor Author

bnchi commented Oct 7, 2024

@wooorm

Sorry I've never worked on a lib before in Rust and I figured I messed up the docs in my initial PR.

Should I close this one then ?

@wooorm
Copy link
Owner

wooorm commented Oct 7, 2024

I think I had all of these in e7ed526, thanks! :)

No worries, you do great work!

@wooorm wooorm closed this Oct 7, 2024
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