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.

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

Removed all .reduce() occurences. #151

Closed
wants to merge 2 commits into from
Closed

Removed all .reduce() occurences. #151

wants to merge 2 commits into from

Conversation

enyo
Copy link

@enyo enyo commented Nov 19, 2012

No description provided.

@enyo
Copy link
Author

enyo commented Nov 19, 2012

Fixes #90

@timoxley
Copy link
Contributor

Why not just shim instead of changing the code?

@enyo
Copy link
Author

enyo commented Nov 20, 2012

I just thought that it's such basic and trivial code, that barely
justifies it's own function. And TJ said to replace it with just a for
loop, so I did that.

But I can create a function for it instead if wanted.

@enyo
Copy link
Author

enyo commented Nov 20, 2012

@visionmedia your thoughts?

@jescalan jescalan mentioned this pull request Dec 10, 2012
@tj
Copy link
Contributor

tj commented Jan 3, 2013

since the client is a component now we can use https://github.com/RedVentures/reduce

@tj
Copy link
Contributor

tj commented Jan 3, 2013

well actually the entire querystring parser thing exists already as a component, ill use that

@tj tj closed this Jan 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants