Skip to content
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

Add support for file contents from other esbuild plugins #52

Merged
merged 4 commits into from
May 10, 2021

Conversation

Arcath
Copy link
Contributor

@Arcath Arcath commented May 10, 2021

As discussed in kentcdodds/mdx-bundler#45 this allows the esbuild plugin to accept file contents either from the disk or from pluginData.contents. Our inMemory plugin will use this to pass mdx content over that doesn't exist on the disk.

test/esbuild.js Show resolved Hide resolved
test/esbuild.js Outdated Show resolved Hide resolved
test/esbuild.js Show resolved Hide resolved
test/esbuild.js Outdated Show resolved Hide resolved
test/esbuild.js Outdated Show resolved Hide resolved
@Arcath
Copy link
Contributor Author

Arcath commented May 10, 2021

I've updated the test. I realised I was going to have to pretty much write mdx-bundler just to run a test with write: false so I've taken the lead from you and got a new test that actually renders the result to confirm it.

Copy link
Owner

@wooorm wooorm left a comment

Choose a reason for hiding this comment

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

all looks good except that one suggestion!

@wooorm wooorm changed the title allow esbuild plugin to accept contents from plugin data Add support for file contents from other esbuild plugins May 10, 2021
@wooorm wooorm merged commit 649c2f0 into wooorm:main May 10, 2021
@wooorm
Copy link
Owner

wooorm commented May 10, 2021

Released in 1.11! Thanks @Arcath!

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