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

Update mdbook, pulldown-cmark and pulldown-cmark-to-cmark dependencies #18

Merged
merged 1 commit into from
Dec 2, 2021

Conversation

tgockel
Copy link
Contributor

@tgockel tgockel commented Dec 1, 2021

This fixes issue #17, which would cause cargo build mdbook-graphviz to
fail. This brings packages up to their current latest versions:

  • pulldown-cmark: From 0.7 to 0.8
  • pulldown-cmark-to-cmark: From 5.0 to 7.1
  • mdbook: From 0.4.x to ^0.4.14

The mdbook update is required because this version added a parameter
named curly_quotes to the new_cmark_parser function. I elected to
set this parameter to false, as there does not appear to be
configuration in the mdbook package for this yet. Presumably, that will
come in the future and it should be copied into this repository.

This fixes issue #17, which would cause `cargo build mdbook-graphviz` to
fail. This brings packages up to their current latest versions:

* pulldown-cmark: From 0.7 to 0.8
* pulldown-cmark-to-cmark: From 5.0 to 7.1
* mdbook: From 0.4.x to ^0.4.14

The mdbook update is required because this version added a parameter
named `curly_quotes` to the `new_cmark_parser` function. I elected to
set this parameter to `false`, as there does not appear to be
configuration in the mdbook package for this yet. Presumably, that will
come in the future and it should be copied into this repository.
@dylanowen dylanowen merged commit cf01327 into dylanowen:master Dec 2, 2021
@dylanowen
Copy link
Owner

Thanks for the quick fix @tgockel I'm kind of surprised that cargo install doesn't use the lockfile by default. Am I right to assume that it'd be a good practice to be more specific with our versions going forward?

@dylanowen dylanowen linked an issue Dec 2, 2021 that may be closed by this pull request
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.

Cargo install mdbook-graphviz fails
2 participants