Blame returning nil when file does not exist in the git tree

[pettomartino]

Related to #240

Instantiating Rugged::Blame with a new file(Rugged::Repository#status == index_new) will throw Rugged::TreeError with does not exist in the given tree.

[doomspork]

LGTM @pettomartino thank you for taking the initiative on this 👍

Any thoughts @mmozuras?

[doomspork]

@pettomartino what is your twitter handle? 😀

[pettomartino]

@doomspork same, @pettomartino

[durnin]

Hello, is this getting merged anytime soon? I'm also trying to use pronto with pre-commit hook using --staged. Thanks!

[doomspork]

@mmozuras any thoughts on this one, is it good to go?

[bjornlinder]

I tested out the changes in my pre-commit hook, didn't see the output I was expecting from pronto. Ideally we'd have pronto linting the newly added (staged) files. Here's what I'm going with for now:

if (( $(git diff --name-only --diff-filter=A HEAD | wc -l) > 0 ))
then
  pronto_output=$(pronto run)
else
  pronto_output=$(pronto run --staged)
fi

[mmozuras]

@doomspork yup, looks good, sorry for taking so long 🙇

@bjornlinder I think that this issue is related to what you wrote about. Would love to read more of your thoughs on how to best approach this 😄.