You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In TopoStats we have separate [project.optional-dependencies] sections for dev, test and pypi as its possible some may want to run tests but not publish to PyPI.
We should perhaps consider splitting the optional dependencies here too so that installs only pull in what is required.
Suggest the following which adds build to pip (required if manually building the package locally rather than relying on CI) and pre-commit to dev.
Caveat - its arguable that if someone is developing tests they should also have dev installed so that code is linted and passes pre-commit.ci, but don't want to force people to adopt this from the off.
The text was updated successfully, but these errors were encountered:
In TopoStats we have separate
[project.optional-dependencies]
sections fordev
,test
andpypi
as its possible some may want to run tests but not publish to PyPI.We should perhaps consider splitting the optional dependencies here too so that installs only pull in what is required.
Suggest the following which adds
build
topip
(required if manually building the package locally rather than relying on CI) andpre-commit
todev
.Caveat - its arguable that if someone is developing tests they should also have
dev
installed so that code is linted and passespre-commit.ci
, but don't want to force people to adopt this from the off.The text was updated successfully, but these errors were encountered: