Skip to content

perf: omit empty ignoreList property from source maps#20319

Merged
alexander-akait merged 2 commits intomainfrom
ignoreList
Jan 19, 2026
Merged

perf: omit empty ignoreList property from source maps#20319
alexander-akait merged 2 commits intomainfrom
ignoreList

Conversation

@xiaoxiaojx
Copy link
Member

@xiaoxiaojx xiaoxiaojx commented Jan 16, 2026

Summary
perf: omit empty ignoreList property from source maps

What kind of change does this PR introduce?
perf

Did you add tests for your changes?
Yes

Does this PR introduce a breaking change?
No

If relevant, what needs to be documented once your changes are merged or what have you already documented?
Nothing

@changeset-bot
Copy link

changeset-bot bot commented Jan 16, 2026

🦋 Changeset detected

Latest commit: b3d27ca

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
webpack Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Jan 16, 2026

This PR is packaged and the instant preview is available (edfaa0f).

Install it locally:

  • npm
npm i -D webpack@https://pkg.pr.new/webpack@edfaa0f
  • yarn
yarn add -D webpack@https://pkg.pr.new/webpack@edfaa0f
  • pnpm
pnpm add -D webpack@https://pkg.pr.new/webpack@edfaa0f

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Let's add a test case where we generate source map and using toMatch() with default generated properties of source map to catch such regressions

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 16, 2026

Merging this PR will not alter performance

✅ 72 untouched benchmarks


Comparing ignoreList (b3d27ca) with main (9a3b428)

Open in CodSpeed

@alexander-akait alexander-akait merged commit edfaa0f into main Jan 19, 2026
50 of 52 checks passed
@alexander-akait alexander-akait deleted the ignoreList branch January 19, 2026 13:29
@alexander-akait
Copy link
Member

Thanks

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.

2 participants