Skip to content

chore: refactor plugin name#19456

Merged
alexander-akait merged 1 commit intomainfrom
refactor-plugin-name
Apr 23, 2025
Merged

chore: refactor plugin name#19456
alexander-akait merged 1 commit intomainfrom
refactor-plugin-name

Conversation

@alexander-akait
Copy link
Copy Markdown
Member

@alexander-akait alexander-akait commented Apr 23, 2025

What kind of change does this PR introduce?

Refactor and move plugin name to own const.

Why?

  • less indentation and better readable
  • ability to reuse plugin name for loggers/caches
  • easy to rename plugin - less mistakes when we forgot to change plugin name in hooks

Did you add tests for your changes?

Existing

Does this PR introduce a breaking change?

No

What needs to be documented once your changes are merged?

No need

@github-actions
Copy link
Copy Markdown
Contributor

Benchmarks:

  • "devtool-source-map": HEAD (2eff6cb) 674 ms ± 49 ms [662 ms; 685 ms] (49 runs)
  • "devtool-source-map": BASE (d988515) 667 ms ± 42 ms [657 ms; 677 ms] (49 runs)
  • "devtool-source-map" change: HEAD (674 ms ± 49 ms [662 ms; 685 ms]) is -1% the same as BASE (BASE) (667 ms ± 42 ms [657 ms; 677 ms])

  • "many-chunks": HEAD (2eff6cb) 1115 ms ± 43 ms [1101 ms; 1128 ms] (31 runs)
  • "many-chunks": BASE (d988515) 1099 ms ± 49 ms [1084 ms; 1114 ms] (32 runs)
  • "many-chunks" change: HEAD (1115 ms ± 43 ms [1101 ms; 1128 ms]) is -1% the same as BASE (BASE) (1099 ms ± 49 ms [1084 ms; 1114 ms])

  • "many-modules": HEAD (2eff6cb) 611 ms ± 29 ms [604 ms; 618 ms] (53 runs)
  • "many-modules": BASE (d988515) 613 ms ± 30 ms [606 ms; 620 ms] (53 runs)
  • "many-modules" change: HEAD (611 ms ± 29 ms [604 ms; 618 ms]) is 0% the same as BASE (BASE) (613 ms ± 30 ms [606 ms; 620 ms])

  • "popular-libraries": HEAD (2eff6cb) 1515 ms ± 33 ms [1503 ms; 1526 ms] (24 runs)
  • "popular-libraries": BASE (d988515) 1562 ms ± 41 ms [1548 ms; 1576 ms] (24 runs)
  • "popular-libraries" change: HEAD (1515 ms ± 33 ms [1503 ms; 1526 ms]) is 3% faster than BASE (BASE) (1562 ms ± 41 ms [1548 ms; 1576 ms])

@alexander-akait alexander-akait merged commit 52dc193 into main Apr 23, 2025
57 of 58 checks passed
@alexander-akait alexander-akait deleted the refactor-plugin-name branch April 23, 2025 12:03
3ru pushed a commit to 3ru/webpack that referenced this pull request Jul 1, 2025
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