Igor Savin

Results 605 comments of Igor Savin

> Create a Migrate_7_To_8.md (suggestion regarding name welcome) Could be `UPGRADE.md` or `MIGRATION.md`

@Fdawgs Isn't it generally preferable to use native solutions over custom libraries?

Streams notoriously change all the time across Node versions :D

@mcollina since it's a test thing and we don't care about performance, can we add nextTick or other delay magic prior to resolving close promise?

@mcollina is it possible to somehow listen to the destroy event from that close? or there is nothing to subscribe to?

would you be open to send a PR for this?

@alaa-eddine Do you accept contributions? I could take a look.

@SukantPal Oh wow, thank you!

@therour Yes, as long as migration guide from knex documentation is followed.

@ericsvendsen Would you be open to send a PR adjusting version in package.json?