appleboy/react-recaptcha

Pre commit broken on Windows.

hbowden opened this issue · 3 comments

hbowden commented

I'm trying to create a few pull requests but the pre-commit hook is broken for me. I looked in the package.json and I see the pre-commit runs lint and test. I ran those commands individually and they both pass but I can't commit with git in this repo. Below is the results of running lint and test then trying to commit. The very bottom of the image is the error message I am receiving.

pre-commit-hook

It appears that pre-commit is broken on Windows: observing/pre-commit#102

Can we remove the pre-commit hook and let CI take care of that? Because this repo is mostly unusable on Windows in it's current state.

andyfry01 commented

@hbowden this was broken for me too, although now I think it should be working. See line 10 and 11 in package.json. Previously pre commit was running test with a --watch flag, which was stalling the commit. Now pre commit runs test without --watch, so I think it will work for you.

hbowden commented

@andyfry01 No, this is still very much broken on Windows. Any ideas @appleboy or @andyfry01 ?

appleboy commented

I think we can remove the pre-commit hooks since we can just run the testing and lint in Travis CI.