vapor-community / node

A formatted data encapsulation meant to facilitate the transformation from one object to another
MIT License
25 stars 20 forks source link

Added defaults to methods asking for context so you don't need to say `in: nil` all the time. #76

Closed Evertt closed 7 years ago

loganwright commented 7 years ago

@Evertt there was a great deal of confusion from end users about how context fit into things because we had the defaults here originally. We debated this quite a bit, and the ultimate decision was that it's better to leave the defaults to the end user if they want to add via extension so that they're aware of the underlying behavior.

I think for now we're going to stick with this decision, perhaps it would be good to include a snippet about how to add this to one's project if they'd like in docs.

Sorry we weren't more clear in the code about this decision, hopefully it wasn't too much effort to add this and maybe somebody will find this and find value in it to add to their own! Thanks as always for being an awesome and active member 👍