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

build(deps): bump prosemirror-trailing-node to 2.0.8 #4949

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LoneRifle
Copy link

Please describe your changes

Bump prosemirror-trailing-node to 2.0.8 to pick up a change that drops @remirror/core-helpers.
The latter package was heavy, bringing in several additional dependencies, but was only used to create
an array of unique elements, and to check for the presence of elements in arrays.

Accordingly, they have been replaced in 2.0.8 with equivalent code that relies on standard built-in objects.

How did you accomplish your changes

npm update prosemirror-trailing-node

How have you tested your changes

N/A - trivial dependency upgrade

How can we verify your changes

Ad-hoc testing in dev

Remarks

This PR is submitted through the course of work done for @opengovsg, whose Starter Kit project is dependent on TipTap

Checklist

  • The changes are not breaking the editor
  • Added tests where possible
  • Followed the guidelines
  • Fixed linting issues

Related issues

remirror/remirror#2245
remirror/remirror#2247

Copy link

netlify bot commented Mar 5, 2024

Deploy Preview for tiptap-embed ready!

Name Link
🔨 Latest commit c2cc471
🔍 Latest deploy log https://app.netlify.com/sites/tiptap-embed/deploys/66442233aced27000946bbbb
😎 Deploy Preview https://deploy-preview-4949--tiptap-embed.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.

@LoneRifle
Copy link
Author

@bdbch @svenadlung - I believe there is a package that is downstream from this one that would benefit from this PR. Are you happy to have a look at this change? It shouldn't change behaviour, and would slightly decrease the total number of dependencies needed for Tiptap.

@bdbch
Copy link
Contributor

bdbch commented Apr 11, 2024

Seems like this PR only changed the package-lock.json but no package.json?

@LoneRifle
Copy link
Author

@bdbch - that's my fault for not understanding the structure of this repo. I'll put a change in when I get the chance.

@LoneRifle
Copy link
Author

@bdbch - updated the PR so that packages/pm/package.json is also changed. Would you be happy to have a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Triage open
Development

Successfully merging this pull request may close these issues.

None yet

2 participants