assign module.exports for CommonJS factory path

mbest / knockout-deferred-updates

Deferred Updates plugin for Knockout <=3.3.0

http://mbest.github.io/knockout-deferred-updates/

134 stars 23 forks source link

assign module.exports for CommonJS factory path #25

Closed ntuckerxx closed 9 years ago

ntuckerxx commented 9 years ago

Is the failure to assign to module.exports an oversight or taking advantage of some peculiarity in the node module loader? It causes the module to return an empty Object in my version of node (0.10.26).

mbest commented 9 years ago

Thanks. This change looks fine.