Skip to content

Async sqlalchemy support#169

Merged
winwinashwin merged 7 commits into
mainfrom
ashwin/contrib-async
May 11, 2026
Merged

Async sqlalchemy support#169
winwinashwin merged 7 commits into
mainfrom
ashwin/contrib-async

Conversation

@winwinashwin
Copy link
Copy Markdown
Contributor

  • docs: Fix typos in docs
  • chore: Remove flexmock from test deps
  • refactor: Remove support for sqlalchemy-i18n
  • chore: Add anyio to core deps and async db drivers to test deps
  • feat(Async SQLAlchemy support #165): Support version traversal with async sqlalchemy

@winwinashwin winwinashwin force-pushed the ashwin/contrib-async branch 6 times, most recently from c962844 to 58aa375 Compare May 11, 2026 10:58
@coveralls
Copy link
Copy Markdown

coveralls commented May 11, 2026

Coverage Report for CI Build 25669055356

Warning

No base build found for commit c508c94 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 96.104%

Details

  • Patch coverage: 305 of 305 lines across 6 files are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 5647
Covered Lines: 5427
Line Coverage: 96.1%
Coverage Strength: 9.55 hits per line

💛 - Coveralls

@winwinashwin winwinashwin force-pushed the ashwin/contrib-async branch from 58aa375 to 1a4fc07 Compare May 11, 2026 11:16
@winwinashwin winwinashwin marked this pull request as ready for review May 11, 2026 11:37
Comment thread pyproject.toml
dependencies = ["SQLAlchemy>=2", "SQLAlchemy-Utils>=0.30.12", "anyio>3"]

[project.optional-dependencies]
asyncio = ["SQLAlchemy[asyncio]>=2"]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I vaguely recall some bug where pip resolution does not consider extras.

So, if you pip install sqlalchemy-history and then pip install sqlalchemy-history[asyncio] does pip just say sqlalchemy is already installed ? or does it install asyncio extra from sqlalchemy ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Tested. pip installs asyncio extra

Comment thread tests/sqlalchemy_async/__init__.py
@winwinashwin winwinashwin force-pushed the ashwin/contrib-async branch from addd517 to 38c2e57 Compare May 11, 2026 12:02
@winwinashwin winwinashwin force-pushed the ashwin/contrib-async branch from 38c2e57 to b133f11 Compare May 11, 2026 12:05
@winwinashwin winwinashwin enabled auto-merge (rebase) May 11, 2026 12:30
@winwinashwin winwinashwin merged commit 5fe88b1 into main May 11, 2026
14 checks passed
@winwinashwin winwinashwin deleted the ashwin/contrib-async branch May 11, 2026 12:43
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.

3 participants