Replace Repo::getTree() with a contrib Tree implementation about gitelephant HOT 1 CLOSED

Hi and thanks for reaching out!

I see both pros and cons to this:

Cons:

• breaking change (fix: major version)
• one full-blown dependency more
• separate discussion possible which package to use
• much work to do (possibility e.g. "adabt" the package implementing TreeishInterface)
• "if it ain't broke, don't fix it"

Pros:

• one thing less to maintain & test here
• a separate implementation might be better known/more familiar to users
• reminder to refactor methods allowing too many input types (example)

I see it as a valid option for a next major version of this library (if @imunhatep does not have major concerns), but not something I would spend time on changing myself.

from gitelephant.