microbiome / mia

Microbiome analysis
https://microbiome.github.io/mia/
Artistic License 2.0
50 stars 27 forks source link

Deprecate mergeRows/mergeCols #392

Closed antagomir closed 1 year ago

antagomir commented 1 year ago

Add an alias as follows

For an example on how to do this, check a similar case for instance in getTopTaxa vs. getTopFeatures.

Deprecate mergeRows/mergeCols with a warning that they have been replaced with another name. See other deprecated functions for an example on how to deprecate an argument name.

Also remember to update manpage examples, vignettes, unit tests in ./tests/

Open an issue to OMA to make corresponding changes there after merging this.

antagomir commented 1 year ago

Suggesting to add some more renaming in this PR as follows:

Some background discussion in #389

TuomasBorman commented 1 year ago

This is done I believe @ake123 ?