front-matter / commonmeta-py

Documentation
https://python.commonmeta.org
MIT License
14 stars 3 forks source link

Documentation dependency `quartodoc` mixed with main dependencies #68

Closed ptamarit closed 3 weeks ago

ptamarit commented 10 months ago

Hello,

We are using commonmeta-py in inveniosoftware/invenio-rdm-records and started running into problems with v0.9.x v0.8.7.

The problem is related to the fact that quartodoc (a documentation dependency), is listed as a main dependency: https://github.com/front-matter/commonmeta-py/blob/a610c1c255f24d6095e9e564a55ead866f3753a8/pyproject.toml#L39-L41

The proper way to define such dependencies according to the Poetry documentation is:

[tool.poetry.group.docs]
optional = true

[tool.poetry.group.docs.dependencies]
# Docs deps here...

Do you think you could release a new version with these dependencies out of the main list?

Thanks, Pablo

mfenner commented 3 weeks ago

Addressed this via e939b70.