Closed sjoro closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.94%. Comparing base (
3c7938c
) to head (25d09b8
). Report is 25 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Totals | |
---|---|
Change from base Build 9513521871: | 0.0% |
Covered Lines: | 51561 |
Relevant Lines: | 53687 |
Did you come up with this idea of listing satpy
as a dependency of itself or is this recommended by some documentation?
@djhoese not sure what you refer to? this one dev = ["satpy[doc,tests]"]
?
Yes that. I've never seen a project reference itself as a dependency.
it should not be a satpy
-dependency (maybe i have misunderstood something then), but this notation in toml, according to what i found online (stackoverflow), is to bind two dependency groups into to one so that pip install -e ".[dev]"
installs the dependecies for doc
and tests
.
https://stackoverflow.com/questions/75490231/pip-pyproject-toml-can-optional-dependency-groups-require-other-optional-depend
what i would like to achieve here is to have a straight forward way to install a dev
dependencies, which to me essentially are the docs
and tests
dependency groups. if the notation is wrong, i can just update the docs and instruct to install doc
and tests
dependencies and leave the toml
-file untouched.
Ok it looks like this is supported in modern versions of pip. I found this thread:
https://discuss.python.org/t/pyproject-toml-optional-dependencies-redundancy-aka-dry-extras/8428/5
And the last 3 comments talk about recursion/inconsistency issues but reference a filed github issue:
https://github.com/pypa/pip/issues/10393
Where it is pointed out that this syntax is supported. That is good enough for me.
This PR fixes documentation under "Developer's Guide" by introducing new dependency group
dev
inpyproject.toml
-file and updating the docs accordingly.Old documentation instructed the user to run
pip install -e ".[all]"
, which only results to a warning as dependency keyall
does not exist inpyproject.toml
-file.