search

classdojo / crisco

MIT License
0 stars 0 forks source link

Function signatures for required middleware callbacks should probably be hash of options with a callback #27

Open cdolivares opened 11 years ago

cdolivares commented 11 years ago

Currently we have something like this,

  crisco.registerMiddleware "verify:featureFlags",
    (CriscoModel, Aux, rootResourceName, resourceSelector, callback) ->

Which isn't ideal because the function's arity varies too much across different required callbacks. Preferable would be to standardize to this

crisco.registerMiddleware "verify:featureFlags", (opts, callback) ->