Skip to content
This repository has been archived by the owner on Jul 29, 2020. It is now read-only.

scan now return len #16

Merged
merged 2 commits into from
Sep 17, 2017
Merged

scan now return len #16

merged 2 commits into from
Sep 17, 2017

Conversation

felipeweb
Copy link
Member

crgimenes
crgimenes approved these changes Sep 15, 2017
View reviewed changes
Copy link
Contributor

@crgimenes crgimenes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

marioidival
marioidival approved these changes Sep 16, 2017
View reviewed changes
Copy link
Contributor

@marioidival marioidival left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@felipeweb felipeweb merged commit 7ee7584 into master Sep 17, 2017
@felipeweb felipeweb deleted the len branch September 17, 2017 23:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@crgimenes crgimenes crgimenes approved these changes

@marioidival marioidival marioidival approved these changes

@avelino avelino Awaiting requested review from avelino

@lerrua lerrua Awaiting requested review from lerrua

@gtsalles gtsalles Awaiting requested review from gtsalles

@cassiobotaro cassiobotaro Awaiting requested review from cassiobotaro

@heltonalves99 heltonalves99 Awaiting requested review from heltonalves99

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@felipeweb @crgimenes @marioidival