Skip to content

fix: add greenlet as an explicit dependency#913

Open
wqxoxo wants to merge 1 commit intocashubtc:mainfrom
wqxoxo:fix/greenlet-explicit-dependency
Open

fix: add greenlet as an explicit dependency#913
wqxoxo wants to merge 1 commit intocashubtc:mainfrom
wqxoxo:fix/greenlet-explicit-dependency

Conversation

@wqxoxo
Copy link

@wqxoxo wqxoxo commented Mar 5, 2026

Fixes #899.

pyproject.toml Outdated
[tool.poetry.dependencies]
python = "^3.10"
SQLAlchemy = {version = "^2.0.35", extras = ["asyncio"]}
greenlet = ">=1.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

greenlet = ">=1.0.0" will install, but it’s a bit broad....perhaps >=3.0.0 or ^3.1.0

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call, updated to >=3.0.0.

SQLAlchemy[asyncio] pulls it in transitively but treats it as optional
on some platforms, so macOS installs fail with "the greenlet library is
required to use this function". Goes in main deps since the mint uses
async SQLAlchemy at runtime.

Fixes cashubtc#899.
@wqxoxo wqxoxo force-pushed the fix/greenlet-explicit-dependency branch from c705067 to cad00eb Compare March 8, 2026 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

Tests fail in newly set up environments

2 participants