medialab / sandcrawler

sandcrawler.js - the server-side scraping companion.

Home Page:http://medialab.github.io/sandcrawler/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Promises in addition to callbacks?

moshewe opened this issue · comments

The done callback and more can be replaced with Promises to make the code more streamlined. Is that something worth a PR or is this not in your development plan?

The thing is I will code this repo from scratch again soon. So if this makes sense to you to use Promises and you happen to code it I will probably merge and release for you but don't get too much attached to the current state of the library :)

Will the API remain somewhat the same, or will it change dramatically?

The concept of the API should somewhat remain the same but some things will change. For instance, I will externalize different engines in different packages so that the code is reusable across multiple engine implementations.

If you want I can start opening an issue concerning the rewrite so I can lay out my plans more precisely and so you can help me think about it.

I'd be glad to!
I want to start using the library and would really love to cooperate.

Did not find much time today to open the issue. Can you ping me tomorrow so I open it please?