suggest rename function remove_constants() to remove_constant_cols()

[avallecam]

remove_constant() is a function whose name could be more explicit to facilitate its recall during analysis. An alternative name may need to consider remove_duplicate which is related but in a different orientation.

The proposed alternative is remove_constant_cols() and, since remove_duplicate_rows()

cc: @Joskerus @lgbermeo

[avallecam]

by looking closer to the reference manual of remove_constants(), this function also remove constant rows. But more specifically, empty rows and columns. Still, are there any alternative names anyone would like to consider?

[Bisaloo]

Note that remove_constants() also has a quasi-conflict with the function it's wrapping: janitor::remove_constant(), which is probably not ideal.

[avallecam]

after meeting with @Karim-Mane in the week this issue was filled, we defined some possible actions. One is to clarify the documentation, like defining a constant (or invariant) column or row and specifying empty columns or rows as a particular case.

If this issue is no longer specific to solve, I agree to close it in favour of another, more specific issue.