-
Notifications
You must be signed in to change notification settings - Fork 14
Description
Hello,
i try to follow your awesome lections but while setting up db i run into error.
Steps to reproduce
clone the repo
$ npm i
$ npm run db:migrate
result
`> nodejs-nights-v2@1.0.0 db:migrate C:\nodejs-open-knowledge
knex migrate:latest --knexfile ./src/database/knexfile.js
C:\nodejs-open-knowledge\node_modules\knex\lib\client.js:74
throw new Error(knex: Required configuration option 'client' is missing.);
^
Error: knex: Required configuration option 'client' is missing.
at new Client (C:\nodejs-open-knowledge\node_modules\knex\lib\client.js:74:11)
at Knex (C:\nodejs-open-knowledge\node_modules\knex\lib\knex.js:57:42)
at initKnex (C:\nodejs-open-knowledge\node_modules\knex\bin\cli.js:95:10)
at Command.commander.command.description.action (C:\nodejs-open-knowledge\node_modules\knex\bin\cli.js:190:17)
at Command.listener (C:\nodejs-open-knowledge\node_modules\commander\index.js:315:8)
at Command.emit (events.js:189:13)
at Command.parseArgs (C:\nodejs-open-knowledge\node_modules\commander\index.js:654:12)
at Command.parse (C:\nodejs-open-knowledge\node_modules\commander\index.js:474:21)
at Liftoff.invoke (C:\nodejs-open-knowledge\node_modules\knex\bin\cli.js:278:13)
at Liftoff.execute (C:\nodejs-open-knowledge\node_modules\liftoff\index.js:203:12)
at module.exports (C:\nodejs-open-knowledge\node_modules\flagged-respawn\index.js:51:3)
at Liftoff. (C:\nodejs-open-knowledge\node_modules\liftoff\index.js:195:5)
at C:\nodejs-open-knowledge\node_modules\liftoff\index.js:165:9
at C:\nodejs-open-knowledge\node_modules\v8flags\index.js:135:14
at C:\nodejs-open-knowledge\node_modules\v8flags\index.js:41:14
at C:\nodejs-open-knowledge\node_modules\v8flags\index.js:53:7`
Expected
DB should migrate
Can you please help me with this issue? I investigated it but it seems that we pass the client property into knex config