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 optional importer
parameter to this.emitFile
#3442
Conversation
5e15861
to
6df327f
Compare
Thank you for your contribution! ❤️You can try out this pull request locally by installing Rollup via
or load it into the REPL: |
Codecov Report
@@ Coverage Diff @@
## master #3442 +/- ##
=======================================
Coverage 95.01% 95.01%
=======================================
Files 171 171
Lines 5841 5843 +2
Branches 1723 1724 +1
=======================================
+ Hits 5550 5552 +2
Misses 157 157
Partials 134 134
Continue to review full report at Codecov.
|
I haven't tested it, but at a glance this is exactly what I was asking for! |
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.
Docs LGTM
Co-Authored-By: Jake Archibald <jaffathecake@gmail.com>
Add optional `importer` parameter to `this.emitFile` (rollup#3442)
Thank you for this! |
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
Resolves #3436
Description
This adds the possibility to specify an importing file when using
this.emitFile
so that relative ids can be resolved correctly. Also cleans up documentation and types to specify thatimporter
can actually beundefined
, in which case the current working directory is used as reference.