Igor Savin

Results 407 comments of Igor Savin

@knownasilya Not that I'm aware of. PRs would be welcome!

@elhigu Was it part of new tarn events?

@kellyrbourg Would you be open to ceate a PR for either 1 or 2? 3 is too intrusive, knex absolutely should not be aware of the schema while building the...

http://knexjs.org/guide/#withuserparams exists for this purpose.

@viqueen Can you also add integration test for this?

Documentation needs to be added here: https://github.com/knex/documentation/pulls

sorry for the delay, I was swamped a bit will take a look over the weekend

> unit test sounds like a good middle ground for now , what do you think ? sure, let's go with a unit test