Skip to content

feat: bump diff from ^5.2.0 to ^7.0.0#5348

Merged
JoshuaKGoldberg merged 1 commit into
mochajs:mainfrom
JoshuaKGoldberg:diff-7
May 19, 2025
Merged

feat: bump diff from ^5.2.0 to ^7.0.0#5348
JoshuaKGoldberg merged 1 commit into
mochajs:mainfrom
JoshuaKGoldberg:diff-7

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg commented May 1, 2025

PR Checklist

Overview

Bumps the production dependency. Some dev dependencies still rely on 4.x or 5.x, so the package-lock.json is a bit bigger now.

See https://github.com/kpdecker/jsdiff/blob/master/release-notes.md for diff's changelog. 6.x had a lot of breaking changes. I couldn't get any to cause bugs here.

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as draft May 1, 2025 17:45
@JoshuaKGoldberg JoshuaKGoldberg changed the title fix: bump diff from ^5.2.0 to ^7.0.0 feat: bump diff from ^5.2.0 to ^7.0.0 May 1, 2025
@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review May 1, 2025 17:55
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team May 1, 2025 18:19
@mark-wiemer
Copy link
Copy Markdown
Member

Works for me, would be great if we could identify and bump the dev deps that rely on diff ^5

@voxpelli
Copy link
Copy Markdown
Member

If someone could do a npm why diff then we can figure that out

@JoshuaKGoldberg JoshuaKGoldberg merged commit 554d6bb into mochajs:main May 19, 2025
78 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the diff-7 branch May 19, 2025 13:45
lennonnikolas pushed a commit to lennonnikolas/mocha that referenced this pull request Jan 24, 2026
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.

📦 Package: Bump diff from 5 to 7

3 participants