senecajs / seneca

A microservices toolkit for Node.js.
http://senecajs.org
MIT License
3.96k stars 312 forks source link

review arrow functions #641

Open rjrodger opened 7 years ago

rjrodger commented 7 years ago

how best to use with Seneca?

lfurzewaddock commented 7 years ago

Kyle Simpson @getify covers this really well here: YDKJS - Arrow Functions.

rjrodger commented 7 years ago

@lfurzewaddock thanks

The issue is that lexical binding is not correct for action definitions, as this is bound to the current Seneca action context (enabling proper action tracing etc).

This also applies to action result callbacks.