Support optional keys to ignore

[eric-hansen]

No description provided.

[tjconcept]

Hi @eric-hansen,

Thanks a lot for your interest in improving this library, and the very comprehensive PR!

I am a bit reluctant to merge this because I generally favour composition of packages over growing package sizes and, as far as I can see, the same behaviour could be reached by combining this package with something like https://www.npmjs.com/package/without-keys:

const diff = require('object-diff')
const wo = require('without-keys')
diff(a, wo(b, [ 'name' ]))

Do you have a use case where this would not work?

[eric-hansen]

I've never seen without-keys package before so not really. I understand your point.

My current use-case for this library just falls under the need of not comparing every key of the object. Saw there was no option for this and figured I'd submit a PR for it. Personally I just prefer not using more libraries than necessary when possible.

If you feel the route you proposed is sufficient enough then that's cool. Just thought I'd toss this your way.

[tjconcept]

Personally I just prefer not using more libraries than necessary when possible.

Absolutely, but, in my opinion, never at the cost of added complexity :)

Thanks again for taking the time.