Skip to content

chore: upgrade markdown-to-jsx#4715

Closed
emilyjablonski wants to merge 2 commits intomainfrom
upgrade/markdown-to-jsx
Closed

chore: upgrade markdown-to-jsx#4715
emilyjablonski wants to merge 2 commits intomainfrom
upgrade/markdown-to-jsx

Conversation

@emilyjablonski
Copy link
Copy Markdown
Collaborator

@emilyjablonski emilyjablonski commented Mar 18, 2025

Related to metrotranscom#1107 and metrotranscom#1126 and metrotranscom#1156

Description

These are the breaking release notes, looks like just Typescript. Our current version was released in 2020.

How Can This Be Tested/Reviewed?

We use markdown-to-jsx to display markdown content. Two example usages to check out are (1) the what to expect page (the first page of an application), and (2) the disclaimer page.

Author Checklist:

  • Added QA notes to the issue with applicable URLs (high level issue is here)
  • Reviewed in a desktop view
  • Reviewed in a mobile view
  • Reviewed considering accessibility
  • Added tests covering the changes
  • Made corresponding changes to the documentation
  • Ran yarn generate:client and/or created a migration when required

Review Process:

  • Read and understand the issue
  • Ensure the author has added QA notes
  • Review the code itself from a style point of view
  • Pull the changes down locally and test that the acceptance criteria is met
  • Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

@emilyjablonski emilyjablonski added dependencies Pull requests that update a dependency file 1 review needed Requires 1 more review before ready to merge labels Mar 18, 2025
@emilyjablonski emilyjablonski changed the title fix: upgrade markdown-to-jsx chore: upgrade markdown-to-jsx Mar 18, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2025

Deploy Preview for bloom-public-seeds ready!

Name Link
🔨 Latest commit 750501c
🔍 Latest deploy log https://app.netlify.com/sites/bloom-public-seeds/deploys/67d9f0163c3b2f0008e1b355
😎 Deploy Preview https://deploy-preview-4715--bloom-public-seeds.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2025

Deploy Preview for partners-bloom-dev ready!

Name Link
🔨 Latest commit 750501c
🔍 Latest deploy log https://app.netlify.com/sites/partners-bloom-dev/deploys/67d9f0162e6bef0008c28d1f
😎 Deploy Preview https://deploy-preview-4715--partners-bloom-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2025

Deploy Preview for bloom-exygy-dev ready!

Name Link
🔨 Latest commit 750501c
🔍 Latest deploy log https://app.netlify.com/sites/bloom-exygy-dev/deploys/67d9f016fe3bf10007efe884
😎 Deploy Preview https://deploy-preview-4715--bloom-exygy-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2025

Deploy Preview for bloom-public-seeds ready!

Name Link
🔨 Latest commit d7ba448
🔍 Latest deploy log https://app.netlify.com/sites/bloom-public-seeds/deploys/67d9f01db163800008b5986c
😎 Deploy Preview https://deploy-preview-4715--bloom-public-seeds.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2025

Deploy Preview for partners-bloom-dev ready!

Name Link
🔨 Latest commit d7ba448
🔍 Latest deploy log https://app.netlify.com/sites/partners-bloom-dev/deploys/67d9f01d9c92b50008d47dbb
😎 Deploy Preview https://deploy-preview-4715--partners-bloom-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 18, 2025

Deploy Preview for bloom-exygy-dev ready!

Name Link
🔨 Latest commit d7ba448
🔍 Latest deploy log https://app.netlify.com/sites/bloom-exygy-dev/deploys/67d9f01d4a702e00086c320c
😎 Deploy Preview https://deploy-preview-4715--bloom-exygy-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Copy Markdown
Collaborator

@ludtkemorgan ludtkemorgan left a comment

Choose a reason for hiding this comment

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

I could be swayed about updating this now and upgrade the other two libraries later. But being on two different major versions could create some issues

Comment thread yarn.lock
tinyqueue "^2.0.3"
vt-pbf "^3.1.3"

markdown-to-jsx@*, markdown-to-jsx@^7.7.4:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

issue: I noticed this is not updating the markdown-to-jsx version but rather pulling in a new version. Looks like ui-seeds and ui-components are still pulling in a 6.11 version

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I haven't been able to find any issues so far, but we should test this in HBA/Doorway to make sure where we use a lot of markdown nothing is broken.

Looking at the dependabot tickets, I don't think they were upgraded due to security issues so I would lean towards upgrading it in ui-seeds and ui-components at the same time so that we are not on two different major versions

@mcgarrye mcgarrye added the needs changes The author must make changes and then re-request review before merging label Mar 24, 2025
@emilyjablonski emilyjablonski added on hold Temporarily paused work and removed 1 review needed Requires 1 more review before ready to merge labels Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file needs changes The author must make changes and then re-request review before merging on hold Temporarily paused work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants