-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Ensure that incremental builds work with 'gulp build'. #5561
Conversation
loganfsmyth
commented
Mar 28, 2017
Q | A |
---|---|
Patch: Bug Fix? | N |
Major: Breaking Change? | N |
Minor: New Feature? | N |
Deprecations? | |
Spec Compliancy? | |
Tests Added/Pass? | |
Fixed Tickets | |
License | MIT |
Doc PR | |
Dependency Changes |
@loganfsmyth, thanks for your PR! By analyzing the history of the files in this pull request, we identified @hzoo and @bcoe to be potential reviewers. |
} else { | ||
srcEx = new RegExp("(packages/[^/]+)/src/"); | ||
libFragment = "$1/lib/"; | ||
function swapSrcWithLib(srcPath) { |
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.
These regexes were gross, so figured this would be easier to follow.
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.
lol much easier 😄
gulp.task("default", ["build"]); | ||
|
||
gulp.task("build", function () { | ||
return gulp.src(scripts) | ||
return gulp.src(scripts, { base: base }) |
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.
This was the core of the problem, we weren't setting a base, so all the filepaths were babel-core/src/index.js
, and the regexes all expected the path to start with packages
.
.pipe(through.obj(function (file, enc, callback) { | ||
gutil.log("Compiling", "'" + chalk.cyan(file.path) + "'..."); | ||
gutil.log("Compiling", "'" + chalk.cyan(file.relative) + "'..."); |
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.
This I just changed since it seems easier to read.
Nice work! |