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.

Sign up for GitHub

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

Jump to bottom

feat: enable Alembic CLI integration #80

Merged

cofin merged 7 commits into litestar-org:main from cofin:alembic-command-tweaks

Oct 1, 2023

Member

cofin commented Sep 30, 2023

Pull Request Checklist

New code has 100% test coverage
(If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
(If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
Pre-Commit Checks were ran and passed
Tests were ran and passed

Description

This PR enables the alembic CLI integration.

Close Issue(s)

cofin added 6 commits

September 26, 2023 13:38


          fix: updated commands

537c803


          fix: additional alembic integration changes

2e76871


          fix: remove old CLI

ea4ef5a


          fix: disable hooks for now

6a140e9


          fix: linting changes

56e36d3


          fix: correct CLI and pin to v0.2.2

28e379c

cofin requested review from a team as code owners

September 30, 2023 03:39

Member Author

cofin commented Sep 30, 2023

@v3ss0n This should get the alembic integration working with fullstack.


          fix: adds ignore for migrations

04c8db2

sonarcloud bot commented Sep 30, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

No Coverage information
0.0% Duplication

Alc-Alc approved these changes

View reviewed changes

src/app/lib/db/migrations/versions/.gitkeep Show resolved Hide resolved

cofin merged commit 6bf83ba into litestar-org:main

6 checks passed

cofin deleted the alembic-command-tweaks branch

October 1, 2023 05:45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment