-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Feature angular material #1649
Feature angular material #1649
Conversation
replace jshint task with standard task in gulp dev formatted all files wrt standard linter fixed lint warnings regarding path and split vars fixed lint warnings around error handling fixed other misc lint warnings
fixed all warnings for standard linter removed jshint tasks removed jshintrc and ignore files removed from package.json
removed bootstrap fixed layout issues fixed header
resolve: { | ||
modulesDirectories: ['bower_components', 'node_modules'] | ||
} | ||
} |
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.
Missing semicolon.
gulp running sass, less before webpack removed all extra css which is not required now
var devCompiler = webpack(myDevConfig); | ||
gulp.task('webpack:build-dev', function(callback) { | ||
var devCompiler = webpack(myDevConfig) | ||
gulp.task('webpack:build-dev', ['sass', 'less'], function (callback) { |
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.
Tried this fix in our project, on a fresh install it prevents webpack from building without having the processed .css files, but it seems to trigger the watch over and over as each .scss file is processed. It's functional, but it creates difficult to unusable output and extends the time for the server to be ready significantly.
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.
I didn't give it a good look earlier. Could you try and fix issue in current master and raise a PR?
WRT: #1648