search

banesullivan / scooby

🐶 🕵️ Great Dane turned Python environment detective
MIT License
47 stars 12 forks source link

Remove `investigate` #17

Closed prisae closed 5 years ago

prisae commented 5 years ago

I am not sure about the rationale behind investigate, as scooby.investigate() and scooby.Report() are identical. And there shouldn't be functions in an __init__.py-file. The only thing to do is move all the docstring from investigate() to Report.

I'll do a PR for that. But then there might have been some thinking/reasoning behind the decision I am not aware...?

prisae commented 5 years ago

It could be made backwards compatible by putting into __init__.py:

from scooby.report import Report as investigate
banesullivan commented 5 years ago

We should just remove it