Skip to content

docs: fix inconsistent ESM and CommonJS syntax Related to #7772 #7923

Merged
alexander-akait merged 4 commits intowebpack:mainfrom
Katari-8055:CommonjsToESM
Mar 6, 2026
Merged

docs: fix inconsistent ESM and CommonJS syntax Related to #7772 #7923
alexander-akait merged 4 commits intowebpack:mainfrom
Katari-8055:CommonjsToESM

Conversation

@Katari-8055
Copy link
Contributor

Summary

What kind of change does this PR introduce?

Did you add tests for your changes?

Does this PR introduce a breaking change?

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

Use of AI

@vercel
Copy link

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-js-org Ready Ready Preview, Comment Mar 6, 2026 7:04am

Request Review

@alexander-akait
Copy link
Member

Looks good, please fix tests, thanks

@Katari-8055
Copy link
Contributor Author

Looks good, please fix tests, thanks

Hi @alexander-akait ,
This test was expecting the CommonJS example (module.exports = {}), but the documentation has been migrated to ESM (export default {}).
Since the copy button should reflect the current code block content, I updated the test assertion to match the new ESM example.

@alexander-akait alexander-akait merged commit b35eb0d into webpack:main Mar 6, 2026
9 checks passed
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