Migrate dev and docs from optional dependencies to PEP 735 dependency groups#956
Migrate dev and docs from optional dependencies to PEP 735 dependency groups#956
dev and docs from optional dependencies to PEP 735 dependency groups#956Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #956 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 39 39
Lines 2691 2691
=========================================
Hits 2691 2691 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
niksirbi
left a comment
There was a problem hiding this comment.
Thanks @lochhh! This was overdue.
At first I was wondering why the test action didn't break, but then I realised because the env for that is installed by tox which is configured in pyproject.toml, so all good.
Should we open the same PR in the python-cookiecutter and at least in our other behavioural repos? (so we don't run the risk of confusing the dev installation syntax across projects).



Description
What is this PR
Why is this PR needed?
Closes #955
What does this PR do?
docsanddevdependencies from "optional dependencies" (extras) to "dependency groups"--groupallgroup that installs bothdocsanddevbuild_sphinx_docs@build-docs-group-dep(from Usedocsgroup dependency from pyproject.toml in build_docs action actions#158)References
The NIU build docs action needs to be updated as well (neuroinformatics-unit/actions#158) as it currently uses the old
[docs]syntax to install docs dependencies.How has this PR been tested?
Is this a breaking change?
No
Does this PR require an update to the documentation?
No
Checklist: