Skip to content

docs(guides): fix package exports example#8046

Open
rolandixor wants to merge 2 commits intowebpack:mainfrom
rolandixor:docs/fix-package-exports-example
Open

docs(guides): fix package exports example#8046
rolandixor wants to merge 2 commits intowebpack:mainfrom
rolandixor:docs/fix-package-exports-example

Conversation

@rolandixor
Copy link
Contributor

Summary

This one addresses #5820 by correcting the example for package exports.

What kind of change does this PR introduce?

Corrected the line:

| package/other-prefix/deep/file.js|.../package/yet-another/deep/file/deep/file.js |

It now correctly has file for package/other-prefix/deep and removes the extra deep/file from the other side.

Did you add tests for your changes?

N/A - docs change.

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?

N/A.

Use of AI

AI was used to identify the correct file to edit and to verify that the manual changes were correct. I take full responsibility for all changes in this PR.

@vercel
Copy link

vercel bot commented Mar 17, 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 18, 2026 10:06pm

Request Review

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.

Each * it is a deep/file, so your change is wrong

Restored /deep/file on the Result column, since that should've been translated from the second `*` wildcard.
@rolandixor rolandixor force-pushed the docs/fix-package-exports-example branch from 1d4772f to a9d7061 Compare March 18, 2026 22:03
@rolandixor rolandixor marked this pull request as ready for review March 18, 2026 22:17
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