-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Build/bundle CLI #706
Build/bundle CLI #706
Conversation
@@ -7,3 +7,4 @@ _actual | |||
coverage | |||
.commithash | |||
.idea | |||
bin/rollup |
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.
Untrack file to ignore.
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.
oops! done, thanks
import { resolve } from 'path'; | ||
import relative from 'require-relative'; | ||
import handleError from './handleError'; | ||
import SOURCEMAPPING_URL from './sourceMappingURL.js'; |
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.
sourceMappingUrl.js
Didn't know where to ask this but at the source. :-) @Rich-Harris curious about this line. Why can't we import? https://github.com/rollup/rollup/blame/master/bin/src/runRollup.js#L6 |
@snuggs IIRC it's because things used to get terribly confusing if you had an import that was relative to the output bundle – Rollup would try and resolve it relative to the source file and all hell would break loose (but if the import was relative to the source file, the |
See #700