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

[refactor] 'build-kit/taskrunner' module #5

Open
kixxauth opened this issue Dec 31, 2010 · 0 comments
Open

[refactor] 'build-kit/taskrunner' module #5

kixxauth opened this issue Dec 31, 2010 · 0 comments

Comments

@kixxauth
Copy link
Member

The taskrunner module was hastily built to bootstrap the Kake project and therefore is not very robust.

Now

  • Do error checking on the .append() method.
  • Abstract the event emitter functionality or use an external notification emitter module (Another event emitter module is under construction elsewhere).
    • Event handlers may be bound after the .run() method and will still be fired OR the .run() method could return a promise.
    • Consider the possibility of running tasks in a future turn of the event loop and do some performance tests to see how much it would slow a large build script.
    • If event binding is used to monitor the 'done' and 'error' events, provide a method for unbinding the handlers.
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

No branches or pull requests

1 participant