Rule suggestion: Suggest sortBy if using orderBy with just asc

[garyking]

Would it be a good rule idea to suggest to use sortBy if using orderBy with just asc?

So _.orderBy(arr, 'text', 'asc') would suggest to be replaced by _.sortBy(arr, 'text') instead, or _.orderBy(arr, ['a', 'b'], ['asc', 'asc']) would suggest to be replaced by _.sortBy(arr, ['a', 'b']) instead.

[ganimomer]

Thanks for the suggestion!
I'm not sure creating an opinionated rule here.
Some people might prefer the opposite to be more explicit, or even to use _.orderBy without the second parameter, which also works. (e.g. _.orderBy(arr, 'text') does the exact same thing as _.sortBy.
In these cases, what's usually done is a rule that accepts a parameter, and the recommended config is opinionated. For example, the chaining rule can either enforce that chains are never used, or that they're always used whenever there's a nested call. (In this case, IIRC the recommended has even actually changed over time).
Maybe a rule called collection-ordering that accepts a first parameter that's either sortBy, orderBy, or orderByExplicit, with the default (and the setting in the recommended config) being sortBy, and enforces that all cases are according to it.
@garyking What do you think?

[garyking]

That sounds good! The idea of using _.orderBy to completely replace _.sortBy by some people is actually interesting, and I might adopt that in my own code, since it makes it easier to change if necessary (by simply adding a third parameter to explicitly specify sort options), and to search among projects.

[ganimomer]

Fixed in v5.0.0.

[garyking]

Just a note, that the rule isn't listed in the readme.