Add object predicate

[SamVerschueren]

Still work in progress

[SamVerschueren]

Implemented all of them. One thing I wasn't sure about, is if we should support dot properties with something like dot-prop. This could be nice for the hasKeys and hasAnyKeys.

Somewhat related to the dot properties. Should we support nested objects in valuesOfType as well then?

[sindresorhus]

Should we make this (and elsewhere) hasAllKeys()? I'm starting to doubt it's clear enough whether it's all keys or any keys if the user hasn't seen hasAnyKeys() yet.

[sindresorhus]

Looking good :)

[sindresorhus]

Implemented all of them. One thing I wasn't sure about, is if we should support dot properties with something like dot-prop. This could be nice for the hasKeys and hasAnyKeys.

Yeah, that could be useful.

Somewhat related to the dot properties. Should we support nested objects in valuesOfType as well then?

What would that look like? I'm not sure I understand.

[SamVerschueren]

What would that look like? I'm not sure I understand.

For instance, this will fail because it only checks the values at root level (Object.keys(input))

const input = {
    foo: 'bar',
    unicorn: {
        rainbow: 'rainbow'
    }
};

ow(input, ow.object.valuesOfType(ow.string));

If we support nested objects, the previous example will succeed.

[sindresorhus]

I think that would be surprising behavior by default, but I can see the use-case for it. So, either a deep option to valuesOfType() or a new method valuesOfTypeDeep(). Which do you prefer?

[SamVerschueren]

I think I'm more in favour of valuesOfTypeDeep() because if we start accepting options, it might look weird.

ow(input, ow.object.valuesOfType(ow.string.minLength(4), {deep: true}));

ow(input, ow.object.valuesOfTypeDeep(ow.string.minLength(4)));

It's also in the same fashion as .equal() and .deepEqual().

[sindresorhus]

👍 valuesOfTypeDeep()