Skip to content

Conversation

@ariporad
Copy link
Contributor

Fixes #249.

api.js Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Naturally, I spend 30 minutes on something to find out that there's already something that does that.

@ariporad ariporad force-pushed the recursive branch 2 times, most recently from 91d178c to db85c03 Compare December 26, 2015 19:21
@ariporad
Copy link
Contributor Author

@sindresorhus: What's the status on this? Is there anything blocking it from getting merged?

@sindresorhus
Copy link
Member

@ariporad At least one other maintainer need to review it.

Next time try to make your PR focused. This should probably have been two separate PRs.

@ariporad
Copy link
Contributor Author

@sindresorhus: sorry about that, I had mentally sorted it by issue, but that doesn't make much sense outside my head. Since I implemented them sequentially, I could split them into seperate PRs, should I do that?

@ariporad
Copy link
Contributor Author

Closing this to split into multiple pull request.

@ariporad ariporad closed this Dec 27, 2015
This was referenced Dec 27, 2015
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.

2 participants