'is' operator

[danschumann]

I am using some code

Users.forge()
  .query('where', 'admin_id', 'is', null)

that works, but now in the new version of knex, it seems to be broken because is isn't an operator and you're checking for operators.

[danschumann]

this broke production!! changes like this are almost too big for patch change and should be a 0.x.0 level change

[tgriesser]

Thats odd because some point operators was indeed being used to check, then it wasn't, which was a regression. The whereNull method is intended for nulls, or if you just omit the is like .query('where', 'admin_id', null) I believe it should it generate the is null.

In any case, I generally try to stick as close as I can to semver when making changes, apologies for the breakage.

[danschumann]

Thanks for quick response time!

[tgriesser]

Sure, cleaned it up a bit so you can also now do .query(col, 'is not', null) and also null can be assumed to be null or 'null' if is or is not are used as the operator. Pushing a release in a few.

[danschumann]

+1