Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add doctest to CI test job #799

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Add doctest to CI test job #799

wants to merge 9 commits into from

Conversation

kadykov
Copy link
Contributor

@kadykov kadykov commented Mar 27, 2024

Fixes #798

@martindurant
Copy link
Member

micromamba can be changed back to miniconda now

@martindurant
Copy link
Member

601     >>> tf = intake.readers.catalogs.TensorFlowDatasetsCatalog().read()
UNEXPECTED EXCEPTION: TypeError("unsupported operand type(s) for |: 'NoneType' and '_UnionGenericAlias'")

suggests that the library (or its etils.epy dependency) isn't actually py38/39 compatible

@kadykov
Copy link
Contributor Author

kadykov commented Mar 27, 2024

Probably there is something wrong with micromamba, because for Python 3.8 it chooses version 1.5.1 that requires-python = ">=3.9", and for Python 3.9 it installs version 1.6.0 that requires-python = ">=3.10".

@martindurant
Copy link
Member

Probably there is something wrong with micromamba

Totally possible, I don't think it's that well supported now that conda uses the mamba solver internally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposal: Add doctest support to testing CI job
2 participants