-
Notifications
You must be signed in to change notification settings - Fork 55
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
Issue #378: Add alembic
support for database migrations
#387
base: master
Are you sure you want to change the base?
Conversation
These should have been removed in 2ee0636, but were missed
Codecov Report
@@ Coverage Diff @@
## master #387 +/- ##
==========================================
- Coverage 75.82% 75.28% -0.54%
==========================================
Files 55 58 +3
Lines 3747 3917 +170
==========================================
+ Hits 2841 2949 +108
- Misses 906 968 +62 |
This PR is still missing tests for the |
In the interest of having the database migration functionality merged sooner, I would suggest that tests for the |
This PR adds
alembic
forpyani
, making it possible to upgrade and downgrade existing databases to accommodate otherwise breaking changes to the ORM, as with the addition offastANI
capabilities.Closes #378.
Type of change
Action Checklist
pyani
repository under your own account (please allow write access for repository maintainers)CONTRIBUTING.md
)pytest -v
non-passing code will not be mergedorigin/master
flake8
andblack
before submissionPull requests
section in thepyani
repository