skill-sample-nodejs-adventure-game icon indicating copy to clipboard operation
skill-sample-nodejs-adventure-game copied to clipboard

Bump require-dir to 0.3.2

Open lazerwalker opened this issue 7 years ago • 4 comments

Hi! Thanks so much for this project — looking forward to poking at it as an example!

Node v8 introduced some breaking changes that break versions of require-dir prior to 0.3.2. On my local dev machine, which runs Node 8, a clean clone and npm install of this repo pulled down an older version of request-dir that crashes on run.

This just bumps the package.json to use a version of require-dir that has the node 8 fix. The more proper solution would be to introduce a lockfile of some sort (whether a yarnfile.lock or package-lock.json), but seeing as this isn't my project I don't want to pick a side on yarn vs npm for you :P.

lazerwalker avatar Jul 27 '17 18:07 lazerwalker

I can confirm that this fixed #42 for me by bumping this version up.

For people using node 8 (which anyone following the instructions and doing a brew install node will have at this point) - this fix is necessary.

ryangardner avatar Aug 05 '17 19:08 ryangardner

this fixed issue #42 (and issue #54) for me as well.

chrmitchell avatar Nov 29 '17 16:11 chrmitchell

@rjamison any chance of getting this merged? It's a very small non-code change that means the project is actually usable in a modern node stack.

lazerwalker avatar Dec 05 '17 01:12 lazerwalker

I'm not sure this repo is being maintained any more. I'm maintaining an updated version with this fix included at samdobson/interactive-adventure-game-tool

samdobson avatar Jan 17 '18 12:01 samdobson