added exported function to compose a handler chain #1660
Conversation
lib/chainComposer.js
Outdated
| @@ -0,0 +1,23 @@ | |||
| 'use strict'; | |||
There was a problem hiding this comment.
Could you move this under a utils and export it like module.exports.utils = require('./utils');.
Would you mind adding a unit test?
|
@DonutEspresso what do you think? should we export it under a namespace like |
|
Either is fine - I don't feel too strongly about it. Let's go with |
|
@hekike Moved/renamed the export and added tests. I can squash if this looks ok. |
|
@paulbakker okay thanks, one small thing: please just call it |
…to export Chain added exported function to compose a handler chain, wihtout the need to export Chain moved chainComposer to helpers added tests for chaincomposer renamed compose export
|
@hekike renamed to |
This compose function makes it possible to create a handler chain, without the need to export Chain publicly. This will be used to implement hot-reloading in the enroute project.
Discussed the idea in person with @hekike.