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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

clarifying status value #20

Merged
merged 1 commit into from Aug 27, 2016

Conversation

Projects
None yet
2 participants
@garu
Contributor

garu commented May 30, 2015

Hello again 馃樅

This is yet another small PR to make sure everyone understands that "success" means "a true value" and "failure" means "a false value". From what I understood of the source code, this is what status() does :)

It fixes this RT issue which is 3 years old 馃檧

Thanks again!

@charsbar

This comment has been minimized.

Show comment
Hide comment
@charsbar

charsbar Apr 21, 2016

Collaborator

@makamaka, please merge this. This is a doc patch.

Collaborator

charsbar commented Apr 21, 2016

@makamaka, please merge this. This is a doc patch.

@charsbar charsbar merged commit 7e53981 into makamaka:master Aug 27, 2016

@charsbar

This comment has been minimized.

Show comment
Hide comment
@charsbar

charsbar Aug 27, 2016

Collaborator

Thanks once again!

Collaborator

charsbar commented Aug 27, 2016

Thanks once again!

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