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
fix(cli): Make gz -a
working again
#573
Conversation
@LinusU if looks good to you then feel free to squash and merge as all CI is passing! |
PS @thierrymichel thanks for the PR and your patience :) |
@jimthedev you're welcome. I hope it will address the issue. |
@@ -6,7 +6,7 @@ export { isClean }; | |||
* Asynchrounously determines if the staging area is clean | |||
*/ | |||
function isClean (repoPath, done) { | |||
exec('git diff --cached --name-only', { | |||
exec('git diff --no-ext-diff --name-only && git diff --no-ext-diff --cached --name-only', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could git diff-index --quiet HEAD
work? 🤔 That way we only have to execute git
once.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LinusU The problem is that HEAD
points to a commit/branch.
So, before your first commit, you will get a fatal: bad revision 'HEAD'
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, right, looks good then 👍 thanks for checking!
ok, released in 3.0.2. Thanks for work @thierrymichel! |
fix #543