Skip to content

Update deps June 2025 #1294

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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Update deps June 2025 #1294

wants to merge 12 commits into from

Conversation

DeeDeeG
Copy link
Member

@DeeDeeG DeeDeeG commented Jun 5, 2025

Testing some dependency bumps to see how they go.

Would be nice to have some of these updated. The various packages/**/package-lock.json changes should be irrelevant to the final Pulsar build, which is based on yarn.lock in the root dir of the project, but it feels better to be on top of those deps nonetheless. Might affect package tests in CI, I suppose? So I'll be looking out for any of that in the CI runs for this PR as well.

This isn't everything I'd like to bump, but it's a start and a fair few commits already, so maybe not a bad thing to break it up. This is what I have available to upload at the moment from my local clone.

@DeeDeeG DeeDeeG changed the title Update deps Jun 2025 Update deps June 2025 Jun 5, 2025
@DeeDeeG

This comment was marked as resolved.

@DeeDeeG
Copy link
Member Author

DeeDeeG commented Jul 21, 2025

Note for reviewers:

These files packages/markdown-preview/package.json, packages/markdown-preview/spec/markdown-preview-spec.js, and yarn.lock should be the only changes actually affecting our build, AFAIK.

The rest of changes (to various package-lock.json files) are to satisfy dependency audit checks that think our individual bundled package dirs' package-lock.json files are used, when in fact they shouldn't be.

(The change in packages/markdown-preview/spec/markdown-preview-spec.js is a trivial change -- a non-semantically meaningful order change in some HTML attributes in a fixed spec string comparison. See commit and hidden comment above for context if desired. I marked the comment as "resolved" since the commit addresses the comment.)

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.

1 participant