Error when commiting about pretty-quick

[jantimon]

Did it also show an error message?

[DavideCarvalho]

Nope, only this

[DavideCarvalho]

Since i’m working inside a client, it’s possible that it’s a network problem, I’ll take a look

[jantimon]

The error happens here:

https://unpkg.com/[email protected]/dist/scms/git.js

const getSinceRevision = exports.getSinceRevision = (directory, { staged, branch }) => {
  try {
    const revision = staged ? 'HEAD' : runGit(directory, ['merge-base', 'HEAD', branch || 'master']).stdout.trim();
    return runGit(directory, ['rev-parse', '--short', revision]).stdout.trim();
  } catch (error) {
    if (/HEAD/.test(error.message) || staged && /Needed a single revision/.test(error.message)) {
      return null;
    }
    throw error;
  }
};

But this is quite unrelated to the last changes we did.

[DavideCarvalho]

Okay, these were my steps

• npm cache clean --force
• remove package-lock.json
• I was using pretty-quick version 1.8, so I updated to 1.10
• deleted node_modules and installed everything again
  Now it's working

[jantimon]

Cool 👍- can we close this issue then?

[azz]

Let's close it. Sounds like git itself exited nonzero with no output. If it props up again we can investigate further.