New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bundle Pure ESM Packages into /vendors/*.js
as CommonJS
#12217
Conversation
9c8c496
to
4717a74
Compare
Can we do this in |
My 2 cents, I personally don't like putting stuff in |
460084c
to
5a86d5f
Compare
We don't have many opportunities to run this script, so I don't think it's worth running it in |
|
||
const changelogUnreleasedDir = fileURLToPath( | ||
new URL("../changelog_unreleased", import.meta.url) | ||
); | ||
|
||
const files = globby.sync(["blog-post-intro.md", "*/*.md"], { | ||
const files = globbySync(["blog-post-intro.md", "*/*.md"], { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Globby accepts URL as cwd
now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually this file is the reason I propose sindresorhus/globby#176
71fb2ba
to
3972a28
Compare
This reverts commit 5a86d5f.
3972a28
to
a2acdd5
Compare
Description
Bundles external Pure ESM Packages to CJS (#12209 (comment)).
See https://github.com/prettier/prettier/blob/b62b8e6d414d10b7cb37c343012a3a89a38aad8a/scripts/vendors/README.md for architecture.
In this PR, bundles following 3 packages:
string-width
(globby
globby
is replaced withfast-glob
in our source code)mem
Checklist
✨Try the playground for this PR✨