hub.js icon indicating copy to clipboard operation
hub.js copied to clipboard

Unable to comment, commit, or push from Bash shell on Windows

Open MikeTschudi opened this issue 5 years ago • 5 comments

Using MINGW64, commands such as npm run c and "C:\Program Files\Git\bin\git.exe" commit -F "C:\...\hub.js\.git\COMMITMESSAGE" fail with

$ npm run c

> @esri/[email protected] c C:\...\hub.js
> npm run precommit && git-cz


> @esri/[email protected] precommit C:\...\hub.js
> ./exit_if_no_staged.sh && lint-staged

'.' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @esri/[email protected] precommit: `./exit_if_no_staged.sh && lint-staged`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @esri/[email protected] precommit script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\...\AppData\Roaming\npm-cache\_logs\2020-10-08T17_16_28_173Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @esri/[email protected] c: `npm run precommit && git-cz`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @esri/[email protected] c script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\...\AppData\Roaming\npm-cache\_logs\2020-10-08T17_16_28_207Z-debug.log

MikeTschudi avatar Oct 08 '20 17:10 MikeTschudi

@drewctate can you remove the bash script from our precommit hook?

tomwayson avatar Oct 16 '20 21:10 tomwayson

Sure. Or maybe I'll just convert it to be a node script.

drewdaemon avatar Oct 17 '20 14:10 drewdaemon

@MikeTschudi next time you pull master could you please confirm that this has been fixed?

tomwayson avatar Oct 19 '20 16:10 tomwayson

@tomwayson I think we can safely close this.

drewdaemon avatar Nov 22 '21 17:11 drewdaemon

Haven't had a commit to test with, but npm run c doesn't fail as shown above; it now fails by not allowing the arrow key to select the type of commit.

MikeTschudi avatar Nov 22 '21 19:11 MikeTschudi