Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

fix(migrations): clean the alembic migrations & scratch revisions #145

Merged
merged 11 commits into from
Mar 30, 2024
Merged

Conversation

frgfm
Copy link
Member

@frgfm frgfm commented Mar 30, 2024

This PR introduces the following modifications:

  • scratch previous migrations
  • clean setup so that revision are properly generated
  • update docker to run migration properly before starting

Copy link

codecov bot commented Mar 30, 2024

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 83.86%. Comparing base (c7fea64) to head (9f04edf).
Report is 1 commits behind head on main.

Files Patch % Lines
src/app/db.py 60.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
- Coverage   83.95%   83.86%   -0.10%     
==========================================
  Files          29       29              
  Lines         916      917       +1     
==========================================
  Hits          769      769              
- Misses        147      148       +1     
Flag Coverage Δ
unittests 83.86% <66.66%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@frgfm frgfm merged commit dc5f8db into main Mar 30, 2024
15 of 17 checks passed
@frgfm frgfm deleted the alembic branch March 30, 2024 02:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant