let setuptools auto detect pkg layout and remove deps#91
Conversation
|
This also fixes downstream issues with doc building for openfe. |
| keywords = ['alchemical free energy', 'setup', 'perturbation', 'network'] | ||
| requires-python = ">= 3.9" | ||
| dependencies = [ | ||
| "numpy", |
There was a problem hiding this comment.
Are we removing this because of gufe or just because we don't want dependencies overall?
One of the thing we'll be doing soon-ish to making gufe optional, so I'm wondering if we should just drop the gufe from this list in the hope that we might be able to put this on pypi? Or just no and we add it back later if needed?
There was a problem hiding this comment.
I vote once we remove the gufe dep we then fill this out correctly (need some min version pins at least) and then get it on pypi
There was a problem hiding this comment.
but for now just have no deps listed
IAlibay
left a comment
IAlibay
left a comment
There was a problem hiding this comment.
Approving early so that we don't block failing things downstream.
2 participants
fixes regression to
src/layout.setuptoolsdidn't detect asrc/layout and created an empty package.This also removes the deps from the
pyproject.tomlwhich are a footgun since it can then install a very old version ofgufe.