-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Major gulp refactor: blueprint.defineTaskGroup() #617
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
Changes from 1 commit
661436c
e52a0f1
bce87b7
7c9755b
fad2c05
b3c09e3
183c315
0044b9b
b13f70c
6952e15
9126e3d
aba0117
aeb0fdd
2629f96
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,14 +8,13 @@ module.exports = (blueprint, gulp) => { | |
| const webpackConfig = require("./util/webpack-config"); | ||
|
|
||
| const docsProject = blueprint.findProject("docs"); | ||
|
|
||
| const configuration = webpackConfig.generateWebpackTypescriptConfig(docsProject); | ||
|
|
||
| gulp.task("webpack-compile-docs", ["docs"], (callback) => { | ||
| gulp.task("webpack-docs", ["docs"], (callback) => { | ||
| webpack(configuration, webpackConfig.webpackDone(callback)); | ||
| }); | ||
|
|
||
| gulp.task("webpack-compile-w-docs", (callback) => { // eslint-disable-line no-unused-vars | ||
| gulp.task("webpack-docs-watch", (callback) => { | ||
| // rely on editor for compiler errors during development--this results in _massive_ speed increase | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we not need the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we are not using eslint anymore! it's all TSLint |
||
| configuration.ts.transpileOnly = true; | ||
| // never invoke callback so it runs forever! | ||
|
|
||
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.
What's the motivation behind this
-gulp-suffixedtslinttask as distinct from the non-suffixedtslinttask? Looks like we'retslint-ing all JS files...and then also JS files withingulp/? Is that necessary on non-typescript files?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.
ok so: the previous iteration of this task ran tslint on all
.jsfiles (hence thepackage/**/*.jsglob).this new iteration runs tslint on the gulp code only because JS files in packages are now linted by
tslint-<package>.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.
Gotcha 👍