husky-4-to-8 icon indicating copy to clipboard operation
husky-4-to-8 copied to clipboard

Update README.md

Open DoctorDerek opened this issue 3 years ago • 3 comments

Hey, I just wanted to leave a note that the README assumes someone's on a *nix system, so the multi-line instructions should probably be written as three separate lines.

After all, most terminals accept multi-line inputs, right? 😀

I updated the README to make this change:

npm install husky@6 --save-dev
npx husky-init
npm exec -- github:typicode/husky-4-to-6 --remove-v4-config

DoctorDerek avatar Apr 16 '21 22:04 DoctorDerek

Also #10 is still open and true (npm exec requires npm >v7) -- I actually updated npm to latest because it didn't make any sense to me that npm exec is a new command, but that's also in the README.

DoctorDerek avatar Apr 16 '21 22:04 DoctorDerek

Finally, I should put out that this PR #18 does not actually provide a great user experience on Windows, but at least it's not such a terrible one.

image

I can understand why you'd be using \ && -- but that particular code is basically unusable on Windows (I have to meticulously look at each line to remove the \).

DoctorDerek avatar Apr 16 '21 22:04 DoctorDerek

Or maybe npx --no-install husky-init would work.

DoctorDerek avatar Apr 16 '21 22:04 DoctorDerek