Remove skycalc_cli from dependencies

[teutoburg]

I don't think this was used anywhere, that's what we have skycalc_ipy for!

Removing this also finally gets rid of setuptools as an indirect dependency.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.58%. Comparing base (6ce5e06) to head (f802342).
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #481   +/-   ##
=======================================
  Coverage   76.58%   76.58%           
=======================================
  Files          66       66           
  Lines        8094     8094           
=======================================
  Hits         6199     6199           
  Misses       1895     1895           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hugobuddel]

Fine with me, but only if you ensure that "readthedocs doesn't include the tests module", or that it is not a problem if that happens anyway.

I just copied that line from 74f98e7d ; I don't know what it actually means.

[teutoburg]

Fine with me, but only if you ensure that "readthedocs doesn't include the tests module", or that it is not a problem if that happens anyway.

The RTD build for this branch looks just fine. Except ofc for the three example notebooks, but that's very much a separate issue that we're already aware of. In the reference API section, which includes all the modules, there is no "test(s)" anywhere to be seen. So I guess we're good on that.

I just copied that line from 74f98e7d ; I don't know what it actually means.

I always wondered about this comment. Maybe I'll ask @astronomyk if he remembers 🙃

But this might actually have been outdated after moving to poetry, or maybe even already after moving to pyprject.toml...