Skip to content

Conversation

@dlespiau
Copy link
Contributor

@dlespiau dlespiau commented Nov 16, 2017

Seems like grepping the output of git status --porcelain broke. Use a
different approach with git diff and only check the events we care about
(having staged and un staged changes).

@dlespiau dlespiau requested a review from bboreham November 16, 2017 17:32
Seems like grepping the output of git status --porcelain broken. Use a
different approach with git diff and only checks the events we care about
(having staged and un staged changes).
@dlespiau dlespiau force-pushed the 2017-11-16-fix-image-tag branch from 9eac7a0 to 2838263 Compare November 16, 2017 17:52
@leth
Copy link
Contributor

leth commented Dec 14, 2017

IIRC the reason I changed this was to avoid git reading all the files to generate a diff, which can be slow over a mounted filesystem.

@bboreham
Copy link
Contributor

bboreham commented Dec 14, 2017

@leth would you care to explain your regex? ISTM that a regex approach should work, but I had git status --porcelain like this:

 M src/kubernetes.js
 M src/server.js
 M tools/image-tag
?? src/oneweave.js
?? src/oneweave.js~

and it decided that meant there was no WIP.

@leth
Copy link
Contributor

leth commented Dec 14, 2017

I tested it with that output, and it correctly said -WIP for me, so I'm not sure what is broken.
I think i've overcomplicated the regex though: grep -qE '^[^ ][^ ]\s' would suffice.

@bboreham
Copy link
Contributor

what is happening with this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants