Corrected paths to node_modules folder used by benchmark

[kliu57]

Closes #970

Code changes:

• The following paths were corrected to point to the actual locations of node_modules:

• Added instructions to Contributing.md to install dependencies for benchmark prior to running it

Screenshot of ./benchmark/benchmark.js successfully running after the fix:

[puzrin]

Use npm run benchmark-deps.

markdown-it/package.json

Line 28 in 8470eb6

"benchmark-deps": "npm install --prefix benchmark/extra/ -g marked@0.3.6 commonmark@0.26.0 markdown-it/markdown-it.git#2.2.1",

[kliu57]

Use npm run benchmark-deps.

markdown-it/package.json

Line 28 in 8470eb6

"benchmark-deps": "npm install --prefix benchmark/extra/ -g marked@0.3.6 commonmark@0.26.0 markdown-it/markdown-it.git#2.2.1",

@puzrin after running npm run benchmark deps, node ./benchmark/benchmark.js still does not work.
I also tried npm install --prefix benchmark/extra/ -g marked@0.3.6 commonmark@0.26.0 markdown-it/markdown-it.git#2.2.1

[puzrin]

Did you try from the project root, as ./benchmark/benchmark.js? Works for me on ubuntu.

[kliu57]

Did you try from the project root, as ./benchmark/benchmark.js? Works for me on ubuntu.

Yes I was at the root folder markdown-it when I ran it with the command node ./benchmark/benchmark.js.

I tried using VS Code terminal and I tried using git bash:

Screenshot from git bash:

[puzrin]

Ok. The current PR is incorrect because benchmarked versions should be installed in the standalone directory as it is now. Probably, something Windows-specific. Unfortunately, can not test Windows now. I'd suggest creating a new issue for the record or creating a new PR, without the intended paths change.

[puzrin]

BTW, last screenshot says can not find benchmark module, make sure regular dependencies are installed (npm i)

[kliu57]

BTW, last screenshot says can not find benchmark module, make sure regular dependencies are installed (npm i)

I have created a new issue as instructed.

Yes, I also tried installing all dependencies using npm i benchmark, npm i commonmark, npm i marked, however it didn't fix the issue (see screenshot in issue showing the attempt).

[kliu57]

Did you try from the project root, as ./benchmark/benchmark.js? Works for me on ubuntu.

By the way, I recently tried from Ubuntu, but it doesn't work for me. Getting that same error message: