-
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
Reworked development gulp - Fixes linnovate/mean#1674 #1682
Reworked development gulp - Fixes linnovate/mean#1674 #1682
Conversation
@ADSKLowenthal I think this fixes #1674 and I cleaned up the code a little to make it easier to follow the flow. |
LGTM on a preliminary glance, however I have not had time to pull and test |
Is Hound-CI complaining about the missing semi-colon at the same time we have some tool/process stripping them? |
We will go back to semicolon i assume. On Wed, Oct 12, 2016 at 8:03 PM, ADSKLowenthal notifications@github.com
Lior Kesos - http://www.linnovate.net |
browser.get('/') | ||
expect(browser.getTitle()).toMatch(/.*MEAN.*/) | ||
browser.get('/'); | ||
expect(browser.getTitle()).toMatch(/.*MEAN.*/); |
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.
'browser' is not defined.
browser.get('/') | ||
expect(browser.getTitle()).toMatch(/.*MEAN.*/) | ||
browser.get('/'); | ||
expect(browser.getTitle()).toMatch(/.*MEAN.*/); |
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.
'browser' is not defined.
describe('Smoke test home page', function () { | ||
it('title should contain MEAN', function () { | ||
browser.get('/') | ||
expect(browser.getTitle()).toMatch(/.*MEAN.*/) | ||
browser.get('/'); |
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.
'browser' is not defined.
describe('Smoke test home page', function () { | ||
it('title should contain MEAN', function () { | ||
browser.get('/') | ||
expect(browser.getTitle()).toMatch(/.*MEAN.*/) | ||
browser.get('/'); |
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.
'browser' is not defined.
tasks: function (changedFiles) { | ||
var tasks = []; | ||
changedFiles.forEach(function (file) { | ||
if (path.extname(file) === '.css' && !~tasks.indexOf('csslint')) tasks.push('csslint') |
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.
Will this be a problem if we need to recompile SASS/LESS?
Reworked development gulp - Fixes #1674, #1690, #1652