Deprecate string.nonempty?

It can be confusing using it with .min when they can both serve the same purpose. For example, it’s impossible to know what happens if I do the following without consulting the source code:

z.string().min(2).nonempty();

Does that remove my minimum length of 2 on .min? The answer is yes, but it’s unclear. I think it would be better to make users just use .min(1) as that is far easier to read and look at.

[colinhacks]

Good call, I agree with this. I’ll probably never remove support but I’ll steer people towards .min(1). Deprecated in #440.

[alii]

As a workaround, you could write .min(1, 'Username must be at least 1 character');