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

Upgrade pulldown-cmark to 0.10.3 #183

Merged
merged 1 commit into from
May 15, 2024
Merged

Conversation

kdarkhan
Copy link
Collaborator

@kdarkhan kdarkhan commented Apr 11, 2024

The latest version of pulldown-cmark-to-cmark now supports pulldown-cmark. This should allow us to upgrade both dependencies.

Had to adjust TagEnd handling and change some types to fix the build failures.

Information on breaking changes in pulldown-cmark is available here. Not sure if I am missing anything. I hope that the tests we have are extensive enough to catch any issues.

@kdarkhan kdarkhan requested a review from mgeisler April 11, 2024 22:01
@codecov-commenter
Copy link

codecov-commenter commented Apr 11, 2024

Codecov Report

Attention: Patch coverage is 85.00000% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 90.27%. Comparing base (39823b5) to head (418eb86).

Files Patch % Lines
i18n-helpers/src/lib.rs 84.21% 0 Missing and 9 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
- Coverage   90.43%   90.27%   -0.16%     
==========================================
  Files          12       12              
  Lines        3010     3034      +24     
  Branches     3010     3034      +24     
==========================================
+ Hits         2722     2739      +17     
  Misses        207      207              
- Partials       81       88       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@mgeisler mgeisler left a comment

Choose a reason for hiding this comment

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

Thanks a lot for tackling this! I thin the footnote handling is off now, but the rest looks great!

i18n-helpers/src/gettext.rs Outdated Show resolved Hide resolved
i18n-helpers/src/lib.rs Show resolved Hide resolved
@kdarkhan kdarkhan force-pushed the pulldown-upgrade branch from 9b8b1d7 to 90f54e7 Compare May 13, 2024 00:01
@kdarkhan kdarkhan changed the title Upgrade pulldown-cmark to 0.10.2 Upgrade pulldown-cmark to 0.10.3 May 13, 2024
Comment on lines 56 to 58
dest_url: _,
title: _,
id: _
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this could be replaced with .. like you do below. It's of course not an important difference 😄

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Agreed, done.

Copy link
Collaborator

@mgeisler mgeisler left a comment

Choose a reason for hiding this comment

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

Thanks a lot @kdarkhan!

@kdarkhan kdarkhan force-pushed the pulldown-upgrade branch from 90f54e7 to 08d10a6 Compare May 15, 2024 12:03
@kdarkhan kdarkhan force-pushed the pulldown-upgrade branch from 08d10a6 to 418eb86 Compare May 15, 2024 12:04
@kdarkhan kdarkhan merged commit 56f60a1 into google:main May 15, 2024
7 checks passed
@kdarkhan kdarkhan deleted the pulldown-upgrade branch May 15, 2024 12:12
klensy added a commit to klensy/mdbook-i18n-helpers that referenced this pull request Jul 2, 2024
html feature was added in google#183, but looks unused
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.

3 participants