Fix memoization for optional args

matthewrudy / memoist

ActiveSupport::Memoizable with a few enhancements

MIT License

920 stars 99 forks source link

Open ashu210890 opened 6 years ago

ashu210890 commented 6 years ago

Fixes #76

matthewrudy commented 6 years ago

@ashu210890 any idea why this breaks all the tests?

which bit isn't backwards compatible?

swrobel commented 5 years ago

@ashu210890 this seems like a worthwhile fix ... are you still interested in getting it merged?