Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Return an enumerator when calling '#each' without a block #219

Merged
merged 1 commit into from
May 31, 2015
Merged

Return an enumerator when calling '#each' without a block #219

merged 1 commit into from
May 31, 2015

Conversation

pabloh
Copy link
Contributor

@pabloh pabloh commented May 31, 2015

Before this PR, when calling each over an spreadsheet would raise an error.
Now it will return an enumerator over all the rows.

@stevendaniels
Copy link
Contributor

Thanks.

stevendaniels added a commit that referenced this pull request May 31, 2015
@stevendaniels
Merge pull request #219 from pabloh/return_enumerator_from_each
971a9d7 
Return an enumerator when calling '#each' without a block
@stevendaniels stevendaniels merged commit 971a9d7 into roo-rb:master May 31, 2015
@pabloh
Copy link
Contributor Author

pabloh commented May 31, 2015

My pleasure

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 93.92% when pulling 6bdb9a1 on pabloh:return_enumerator_from_each into 84982e0 on roo-rb:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 93.92% when pulling 6bdb9a1 on pabloh:return_enumerator_from_each into 84982e0 on roo-rb:master.

@dlanner dlanner mentioned this pull request Jun 9, 2015
Merged
@pabloh pabloh deleted the return_enumerator_from_each branch July 11, 2015 01:42
@stevendaniels stevendaniels mentioned this pull request Jul 18, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pabloh @stevendaniels @coveralls