How to add an option for custom contents of file.

[gucong3000]

I'm planning to use ' git-blame' instead of my js file, but I need to customize contents of file.

[gucong3000]

This is an example used in gulp:

var toReadStream = require('spawn-to-readstream'),
    spawn        = require('child_process').spawn;

/**
 * Get what revision and author last modified each line of a file
 * @param   {vinyl}    file    `vinyl` file (https://github.com/gulpjs/vinyl)
 */
function blame (file) {
  var args = ['blame', '-l', '-t', '-w', '-C', '-M'];
  if (file.contents) {
    args.push('--contents');
    args.push('-');
  }
  args.push('--');
  args.push(file.path);

  var blame = spawn('git', args, {
    cwd: file.cwd
  });

  if (file.isBuffer()) {
    blame.stdin.write(file.contents);
    blame.stdin.end();
  } else if (file.isStream()) {
    file.contents.pipe(blame.stdin);
  }

  return toReadStream(blame);
}

[alessioalex]

Hi @gucong3000 , not sure I understand. This module parses the output of the git blame command, what exactly do you mean by custom contents of file?

As for getting the corrects results after modifying a document (the issue opened in vscode-gitblame repo), I think you need to keep track of git blame data and see what's changed in there.

[gucong3000]

In this animation, the last line of code is not modified, but the blame is lost because of the newline that was inserted earlier

[alessioalex]

I see. This should be handled in vscode-gitblame though, it's not an issue of git-blame though.

[gucong3000]

git-blame should provide a parameter to vscode-gitblame to pass the unsaved code

[alessioalex]

That's outside of the scope of this module in my opinion. vscode-gitblame should keep track of the edited lines instead.

[gucong3000]

Only need a little change, can I push a PR to demonstrate my intention?

[alessioalex]

Ok

[gucong3000]

#8
https://git-scm.com/docs/git-blame#git-blame---contentsltfilegt

[alessioalex]

I understand. Thank you for the pull requests. I'll take a look over them when I have some time. Also, please leave the linting as it is and don't add standard or other stuff. Those can be addressed in other pull requests.