Skip to content
This repository has been archived by the owner on Mar 26, 2018. It is now read-only.

Make gulp work properly #1164

Merged
merged 5 commits into from
Aug 19, 2015
Merged

Make gulp work properly #1164

merged 5 commits into from
Aug 19, 2015

Conversation

stevemao
Copy link
Contributor

typo
add default task
add missing callback arg
Also change cb to callback to make it consistent.
switch to libsass
Ruby sass is not working like this.

@sindresorhus
Copy link
Member

Needs a proper title and description.

@stevemao stevemao changed the title gulp fix Make gulp work properly Aug 19, 2015
@stevemao
Copy link
Contributor Author

@sindresorhus not sure about the title. gulp does not work at all so basically I'm trying to fix it.

gulp.task('clean:tmp', function (cb) {
rimraf('./.tmp', cb);
gulp.task('clean:tmp', function (callback) {
rimraf('./.tmp', callback);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No thanks. cb is the convention name in Node.

Also change `callback` to `cb` to make it consistent.
Ruby sass is not working like this.
eddiemonge added a commit that referenced this pull request Aug 19, 2015
@eddiemonge eddiemonge merged commit 52540a7 into yeoman:master Aug 19, 2015
@stevemao
Copy link
Contributor Author

Still problems and some depends on what tools and what configs you want to use.

@arthurvr arthurvr mentioned this pull request Oct 5, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants