chore: rename innerJoin to intersectionWith

[adispring]

innerJoin is a sql command, R.innerJoin only has inner but no join, so rename it to intersectionWith may be more accurate?

[customcommander]

Makes sense to me. Of course that's a breaking change ;) Question: did you rename the test file?

[adispring]

@customcommander , Yes, thanks for remind.

[customcommander]

LGTM and makes perfect sense. Would like another approval just in case. Otherwise let's merge this end of week latest?

[customcommander]

@CrossEye Any objection?

[CrossEye]

I agree with the change.

I guess ,,we need to decide whether 1.0 or 0.29 is next. When we rename a function, we usually keep the old name around as an alias for a minor version. If we're building 0.29 I would do so, but I think if we're headed right for 1.0, then we need to skip it this time.

[customcommander]

@CrossEye: I wouldn't bother with 0.29 at this point and would rather keep the momentum for 1.0.0 and ship as many breaking changes in that one.