Skip to content
This repository has been archived by the owner on Mar 25, 2022. It is now read-only.

Bump dependency on commonmark to >= 0.8.1 #147

Merged
merged 1 commit into from Mar 19, 2019
Merged

Bump dependency on commonmark to >= 0.8.1 #147

merged 1 commit into from Mar 19, 2019

Conversation

loganrosen
Copy link
Contributor

In #118, there was a change made to rename all commonmark imports from commonmark to CommonMark due to a breaking change to the package name in that dependency in version 0.8.1. However, the dependency on commonmark wasn't updated, which means that people pulling in commonmark >= 0.7.3 and < 0.8.1 are experiencing issues while using recommonmark. This updates the versioned dependency accordingly to resolve this issue.

In #118, there was a change made to rename all commonmark imports from `commonmark` to `CommonMark` due to a breaking change to the package name in that dependency in version 0.8.1. However, the dependency on `commonmark` wasn't updated, which means that people pulling in commonmark >= 0.7.3 and < 0.8.1 are experiencing issues while using recommonmark. This updates the versioned dependency accordingly to resolve this issue.
Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for the update!

@ericholscher ericholscher merged commit 815d75e into readthedocs:master Mar 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants