Fix _.maxBy and _.minBy

[gu-xionghong]

In the master, the _.maxBy and _.minBy will work like this:

const objects = [{ n: 1 }, { n: 3 }, { n: 2 }];

_.maxBy(objects, ({n}) => n)
// expect {n: 3}, but return {n: 2}

_.minBy(objects, ({n}) => n)
// expect {n: 1}, but return {n: 2}

[jdalton]

Thanks @gu-xionghong!

[falsyvalues]

Does tests pass before this change? Maybe we should add more if so.

[gu-xionghong]

Hi, @falsyvalues , I found that after this commit which removes the baseIteratee, the master branch can't be passed the tests. But the other branches always keep the different version, and they can be passed the tests.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.