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

synchronous run function? #14

Closed
MichaelCMCG opened this issue Oct 27, 2016 · 2 comments
Closed

synchronous run function? #14

MichaelCMCG opened this issue Oct 27, 2016 · 2 comments

Comments

@MichaelCMCG
Copy link

Can you please add a synchronous run function? I am not making any I/O calls and everything is in memory in node so your parallel processing provides no benefit and it greatly complicates implementation.
Or provide me with access your github and I will do it.

@CacheControl
Copy link
Owner

The promise-based api is fairly baked in at this point; a synchronous run would be non-trivial to add. This doesn't mean it can't be done, simply that we need to keep the effort required proportional to the value it would provide. Could you describe or perhaps show an example of exactly how this is complicating your implementation?

@CacheControl
Copy link
Owner

CacheControl commented Nov 2, 2016

Closing this for now. If you have an opportunity to elaborate this, feel free to reopen; I'd be interested to hear your thoughts.

I should note that if it's simply the syntax of Promises that is causing concern, you might take a look at using an async/await polyfill, which are easy to install these days and even natively available in some browsers.

let results = await engine.run()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants