Ambiguity on docs for .set()

[thanpolas]

As per the docs:

.set(key, value, [expiraiton], [refresh(key, cb)], [callback])

expiration and refresh are optional, so effectively one could write:

cs.set('key', 'value', function(err, res) {});

However that is not the case as the callback will never be invoked.

[jpodwys]

This is a problem with the code not the documentation. It appears that I'm not assigning the callback when only 3 arguments are passed.

[jpodwys]

It seems I spoke too soon. Because both are functions and both accept they same number of arguments, I can't determine which is which. I'll updates the docs.

[jpodwys]

I created a branch to try to fix the issue you listed above both by adjusting the code and by adding documentation. Please have a look at the diff and let me know what you think.

[thanpolas]

yup, that's what should be done for this case... Let's continue this discussion on jpodwys/cache-service#19

[jpodwys]

.set() fixed and readme updated in 1.3.6.