Skip to content
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

Update gulp API doc with info on file base #700

Merged
merged 1 commit into from
Sep 27, 2014
Merged

Conversation

demisx
Copy link
Contributor

@demisx demisx commented Sep 26, 2014

Expand on what file base is and how it affects the relative path calculation. Fixes #699.

@stringparser
Copy link

Niice! to me it's very clear. Let's see what @contra says.

// write path: '.build/dir2/dir3/whatever.txt'
```

On the other hand, when `gulp.src` is used, for instance, inside of a callback where it is
Copy link
Member

Choose a reason for hiding this comment

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

shouldnt mention callbacks, TMI

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The reason I've shown callback here was because this was the real situation I had run into when gulp.src was dealing with string paths instead of glob. I can rework the example into a contrived one and pass a string path directly instead of a callback:

gulp.src('app/css/somefile.scss')
  .pipe(gulp.dest('build'));

Let me know if this is preferred.

Copy link
Member

Choose a reason for hiding this comment

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

Just saying it seems like too much information and doesn't really have anything to do with the API itself

@stringparser
Copy link

Condensed what @demisx wrote, and made the consistency changes. PR on your fork @demisx

@stringparser
Copy link

I'm thinking, shouldn't this be below just like the other options?
See here from my fork.

@demisx
Copy link
Contributor Author

demisx commented Sep 27, 2014

@contra There are 8 commits in this PR. Shall I rebase them against master into one?

@yocontra
Copy link
Member

yeah, squash them down

@demisx
Copy link
Contributor Author

demisx commented Sep 27, 2014

Done. Please review.

yocontra pushed a commit that referenced this pull request Sep 27, 2014
Update gulp API doc with info on file base
@yocontra yocontra merged commit 22979d1 into gulpjs:master Sep 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect relative path in gulp.dest() > vinyl-fs
3 participants