New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New boolean comparison functions #44

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@dionys

dionys commented Jul 30, 2016

The true and false comparicon functions added as described in #41.

@rjbs

This comment has been minimized.

Show comment
Hide comment
@rjbs

rjbs Apr 19, 2018

Owner

I'm not applying this as is, at least yet. true and false are too likely to be in use (I think) to just start putting them into the v0 export set everywhere. bool(1) and bool(0) are tolerable substitutes for now. Adding them as non-defaults is more of a maybe. Then you could say:

use Test::Deep ':v0', 'true', 'false';

...but is that really better than bool(1) and bool(0)? I'm not sure.

Owner

rjbs commented Apr 19, 2018

I'm not applying this as is, at least yet. true and false are too likely to be in use (I think) to just start putting them into the v0 export set everywhere. bool(1) and bool(0) are tolerable substitutes for now. Adding them as non-defaults is more of a maybe. Then you could say:

use Test::Deep ':v0', 'true', 'false';

...but is that really better than bool(1) and bool(0)? I'm not sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment