Skip to content

Update example quick start sequence to work #3

Closed
wants to merge 1 commit into from

2 participants

@nickcarenza

$>node

var qc = require("quickcheck");
Error: Cannot find module 'quickcheck'
at Function.Module._resolveFilename (module.js:338:15)
at Function.Module._load (module.js:280:25)
at Module.require (module.js:364:17)
at require (module.js:380:17)
at repl:1:10
at REPLServer.self.eval (repl.js:110:21)
at repl.js:249:20
at REPLServer.self.eval (repl.js:122:7)
at Interface. (repl.js:239:12)
at Interface.EventEmitter.emit (events.js:95:17)

@nickcarenza nickcarenza Update example quick start sequence to work
$>node
> var qc = require("quickcheck");
Error: Cannot find module 'quickcheck'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at repl:1:10
    at REPLServer.self.eval (repl.js:110:21)
    at repl.js:249:20
    at REPLServer.self.eval (repl.js:122:7)
    at Interface.<anonymous> (repl.js:239:12)
    at Interface.EventEmitter.emit (events.js:95:17)
638afb4
@mcandre
Owner
mcandre commented Nov 28, 2013

I'm going to reject this change. If this file were meant as part of the library, I would use ./ in the require statement. But the example is intended as a literal you could drop into any context, not just working in the same directory as the quickcheck source code.

The example assumes quickcheck has been installed.

@mcandre
Owner
mcandre commented Nov 28, 2013

If you instead modified the pull request to say in the README that one should npm install quickcheck and then run the example code, then I would accept the pull request.

@nickcarenza nickcarenza closed this Oct 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.