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

Already on GitHub? Sign in to your account

Use 'empty?' instead of 'any?' and 'none?' to match 'nil' and 'false' #187

Merged
merged 1 commit into from Nov 9, 2012

Conversation

Projects
None yet
3 participants
Contributor

nanocity commented Nov 9, 2012

No description provided.

Contributor

alindeman commented Nov 9, 2012

This looks good to me. @myronmarston, @dchelimsky, see any weaknesses?

Owner

myronmarston commented Nov 9, 2012

Looks great. Nice catch, @nanocity!

alindeman added a commit that referenced this pull request Nov 9, 2012

Merge pull request #187 from nanocity/yield_arguments
Use 'empty?' instead of 'any?' and 'none?' to match 'nil' and 'false'

@alindeman alindeman merged commit f8a8065 into rspec:master Nov 9, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment