Skip to content

auto migrate#472

Merged
ericbuckley merged 4 commits into
mainfrom
471/auto-migrate
Jul 17, 2025
Merged

auto migrate#472
ericbuckley merged 4 commits into
mainfrom
471/auto-migrate

Conversation

@ericbuckley

@ericbuckley ericbuckley commented Jul 17, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

Replace INITIALIZE_TABLES config value with AUTO_MIGRATE. The latter also includes the ability to apply pending alembic migrations if they exist.

Related Issues

closes #471

<--------------------- REMOVE THE LINES BELOW BEFORE MERGING --------------------->

Checklist

Please review and complete the following checklist before submitting your pull request:

  • I have ensured that the pull request is of a manageable size, allowing it to be reviewed within a single session.
  • I have reviewed my changes to ensure they are clear, concise, and well-documented.
  • I have updated the documentation, if applicable.
  • I have added or updated test cases to cover my changes, if applicable.
  • I have minimized the number of reviewers to include only those essential for the review.

Checklist for Reviewers

Please review and complete the following checklist during the review process:

  • The code follows best practices and conventions.
  • The changes implement the desired functionality or fix the reported issue.
  • The tests cover the new changes and pass successfully.
  • Any potential edge cases or error scenarios have been considered.

@ericbuckley ericbuckley self-assigned this Jul 17, 2025
@ericbuckley ericbuckley added the automation Automation of processes label Jul 17, 2025
@codecov

codecov Bot commented Jul 17, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.42%. Comparing base (ff50edf) to head (197363a).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/recordlinker/database/__init__.py 93.75% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #472      +/-   ##
==========================================
- Coverage   98.46%   98.42%   -0.04%     
==========================================
  Files          41       41              
  Lines        2407     2412       +5     
==========================================
+ Hits         2370     2374       +4     
- Misses         37       38       +1

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ericbuckley ericbuckley marked this pull request as ready for review July 17, 2025 00:29
@ericbuckley ericbuckley merged commit 7e23148 into main Jul 17, 2025
15 checks passed
@ericbuckley ericbuckley deleted the 471/auto-migrate branch July 17, 2025 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bamader bamader bamader approved these changes

@m-goggins m-goggins Awaiting requested review from m-goggins m-goggins is a code owner

Assignees

@ericbuckley ericbuckley

Labels

automation Automation of processes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

auto migrate

2 participants

@ericbuckley @bamader