Allow tag to accept a stream

[jasonkarns]

Allow tag to accept a stream

[stephenlacy]

Is there any way git tag would accept files?
If not it should not be a stream, as there is no reason for a vinyl source.

[jasonkarns]

The idea wouldn't be for the files to go to git tag but be used for the tag. Though the more I think about it though, the less I think this sugar should exist at this level. Better for this to be a pure 1 to 1 mapping to git and for added sugar to be added via other modules.

[eseceve]

What about if I want to do a gulp task like this?:

var git = require('gulp-git');
gulp.task('myTask', function() {
  return gulp.src(['/my/files.ext'])
    .pipe(git.add())
    .pipe(git.commit('my message')
    .pipe(git.tag(myVersion, 'my message')
    .pipe(git.push('origin', 'master', {args: '--tags'}));
});

@eseceve
Did you manage to do that?
I always get the previous commit tagged and pushed. But the current commit is not pushed.

I'm getting this error message:

TypeError: undefined is not a function"

which ties in this line:

.pipe(git.tag(v, message)) is undefined...

[stephenlacy]

@dreadcast Tag is not a stream, and won't be changed to a stream.
https://github.com/stevelacy/gulp-git/blob/master/examples/gulpfile.js#L64

use

var git = require('gulp-git');
gulp.task('myTask', function() {
  return gulp.src(['/my/files.ext'])
    .pipe(git.add())
    .pipe(git.commit('my message')
    .on('end', function() {
      git.tag('v1.1.1', 'Version message', function (err) {
        //if (err) ...
        git.push('origin', 'master', {args: '--tags'});
      });
    });
});

I rolled gulp-git back to version 0.2.0 and @eseceve example almost works (as @bevacqua 's contra)... I could add, commit and tag the latest commit and finally get push() to work :(
Except that push option is a string '--tags'

@stevelacy
Thank for your example, I will try it tomorrow and then use the latest gulp-git version again !

[eseceve]

@dreadcast https://gist.github.com/eseceve/1461a561b4184dd6e2da

@eseceve
I got server errors with @stevelacy 's example then I digged into your recipe and it works great!