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

chore: unpin Advanced Alchemy #3491

Merged
merged 16 commits into from May 13, 2024
Merged

chore: unpin Advanced Alchemy #3491

merged 16 commits into from May 13, 2024

Conversation

cofin
Copy link
Member

@cofin cofin commented May 11, 2024

Description

This PR unpins Advanced Alchemy.

This allows version 0.9.3 to be installed with teh litestar[sqlalchemy] command group.

This enables the following:

  • A numebr of repository and service optimizations and fixes
  • SQLAlchemyQueryRepository from advanced alchemy
  • to_schema from the Service used there (see the fullstack repository for usage details)

However, this AA release also had to integrate it's own copies of the Litesatr OffsetPagination dataclass. Litestar will attempt to use this version of the dataclass when a supported version is installed. This is implemented similar to how the filters in filters.py work currently.

Closes

@cofin cofin requested review from a team as code owners May 11, 2024 17:47
@github-actions github-actions bot added area/dependencies This PR involves changes to the dependencies size: small type/feat pr/internal labels May 11, 2024
@cofin cofin changed the title feat: unpin Advanced Alchemy chore: unpin Advanced Alchemy May 11, 2024
pyproject.toml Outdated Show resolved Hide resolved
@github-actions github-actions bot added area/docs This PR involves changes to the documentation area/plugins This PR involves changes to the plugins labels May 11, 2024
@peterschutt
Copy link
Contributor

For the 3.8 failure, you'd either have to update the AA pagination class to use the typing generic collection types (ie List not list), or restrict AA to a lower version for python 3.8 installs.

@cofin
Copy link
Member Author

cofin commented May 11, 2024

Thanks - Just saw this as I came to the same conclusions.

Resolved in litestar-org/advanced-alchemy#191

Copy link

codecov bot commented May 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.30%. Comparing base (0ecb245) to head (5ee32e9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3491      +/-   ##
==========================================
- Coverage   98.30%   98.30%   -0.01%     
==========================================
  Files         328      328              
  Lines       14884    14875       -9     
  Branches     2366     2365       -1     
==========================================
- Hits        14632    14623       -9     
  Misses        115      115              
  Partials      137      137              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

litestar/plugins/sqlalchemy.py Outdated Show resolved Hide resolved
litestar/plugins/sqlalchemy.py Outdated Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
cofin and others added 2 commits May 13, 2024 11:35
Co-authored-by: Peter Schutt <peter.github@proton.me>
Copy link

sonarcloud bot commented May 13, 2024

Copy link

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/3491

@cofin cofin merged commit 576beb6 into main May 13, 2024
28 checks passed
@cofin cofin deleted the aa-unpin branch May 13, 2024 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies This PR involves changes to the dependencies area/docs This PR involves changes to the documentation area/plugins This PR involves changes to the plugins pr/internal size: small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants