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

Extracted compatibility module to work without AioModel subclassing #232

Merged
merged 5 commits into from
May 9, 2024

Conversation

rudyryk
Copy link
Member

@rudyryk rudyryk commented May 4, 2024

Extracted module peewee_async_compat which contains Manager and other older interfaces to make cleaner separation from newer implementation based on AioModel etc.

  • Implemented patching queries for compatibility, check the _patch_query_with_compat_methods()
  • Added tests for models and queries without AioModel subclassing
  • Removed _swap_database(), it was an internal purely experimental feature
  • General minor cleanups

peewee_async_compat.py Outdated Show resolved Hide resolved
@rudyryk rudyryk force-pushed the refactoring/manager-compat branch from a0138ba to 06386fa Compare May 5, 2024 17:39
@rudyryk rudyryk changed the base branch from example/aiohttp to master May 5, 2024 17:40
@rudyryk rudyryk requested a review from kalombos May 9, 2024 15:40
peewee_async.py Show resolved Hide resolved
@rudyryk rudyryk force-pushed the refactoring/manager-compat branch from bd04d1c to e12b10c Compare May 9, 2024 17:17
@rudyryk rudyryk merged commit 72f033f into master May 9, 2024
5 checks passed
@rudyryk rudyryk deleted the refactoring/manager-compat branch May 9, 2024 17:46
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.

2 participants