Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,10 @@ jobs:
run: uv sync --frozen --all-extras --dev

- name: Run Ruff linter
run: uv run ruff check .
run: uv run --frozen ruff check .

- name: Run Ruff formatter check
run: uv run ruff format --check .
run: uv run --frozen ruff format --check .

typecheck:
name: Type Check
Expand All @@ -74,10 +74,10 @@ jobs:
run: uv sync --frozen --all-extras --dev

- name: Run MyPy
run: uv run mypy app/
run: uv run --frozen mypy app/

- name: Run Pyright
run: uv run pyright app/
run: uv run --frozen pyright app/

test:
name: Test
Expand Down Expand Up @@ -118,13 +118,13 @@ jobs:
- name: Run migrations
env:
DATABASE_URL: postgresql+asyncpg://forecastlab:forecastlab@localhost:5432/forecastlab_test
run: uv run alembic upgrade head
run: uv run --frozen alembic upgrade head

- name: Run tests
env:
DATABASE_URL: postgresql+asyncpg://forecastlab:forecastlab@localhost:5432/forecastlab_test
APP_ENV: testing
run: uv run pytest -v --tb=short
run: uv run --frozen pytest -v --tb=short

migration-check:
name: Migration Check
Expand Down Expand Up @@ -165,12 +165,12 @@ jobs:
- name: Apply migrations to fresh DB
env:
DATABASE_URL: postgresql+asyncpg://forecastlab:forecastlab@localhost:5432/forecastlab_migration_test
run: uv run alembic upgrade head
run: uv run --frozen alembic upgrade head

- name: Verify no pending migrations
env:
DATABASE_URL: postgresql+asyncpg://forecastlab:forecastlab@localhost:5432/forecastlab_migration_test
run: |
# Check that current head matches database
uv run alembic current
uv run --frozen alembic current
# This would fail if there are unapplied migrations
12 changes: 6 additions & 6 deletions .github/workflows/phase-snapshot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,24 +58,24 @@ jobs:
- name: Lint check
id: lint
run: |
uv run ruff check .
uv run ruff format --check .
uv run --frozen ruff check .
uv run --frozen ruff format --check .

- name: Type check
id: typecheck
run: |
uv run mypy app/
uv run pyright app/
uv run --frozen mypy app/
uv run --frozen pyright app/

- name: Run migrations
id: migration
run: uv run alembic upgrade head
run: uv run --frozen alembic upgrade head

- name: Run tests
id: test
env:
APP_ENV: testing
run: uv run pytest -v --tb=short
run: uv run --frozen pytest -v --tb=short

create-snapshot:
name: Create Audit Snapshot
Expand Down
18 changes: 9 additions & 9 deletions .github/workflows/schema-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -59,14 +59,14 @@ jobs:
- name: Fresh DB migration test
run: |
echo "::group::Applying all migrations to fresh database"
uv run alembic upgrade head
uv run --frozen alembic upgrade head
echo "::endgroup::"

- name: Check migration chain integrity
run: |
echo "::group::Verifying migration chain"
# Get all revision heads - should be exactly one
HEADS=$(uv run alembic heads 2>&1)
HEADS=$(uv run --frozen alembic heads 2>&1)
HEAD_COUNT=$(echo "$HEADS" | grep -c "^[a-f0-9]" || true)

if [ "$HEAD_COUNT" -gt 1 ]; then
Expand All @@ -84,7 +84,7 @@ jobs:
echo "::group::Checking for schema drift"
# alembic check compares models to current DB state
# Returns non-zero if autogenerate would create new migrations
if uv run alembic check 2>&1; then
if uv run --frozen alembic check 2>&1; then
echo "Schema is in sync with models"
else
echo "::error::Schema drift detected - models don't match migrations"
Expand All @@ -97,7 +97,7 @@ jobs:
run: |
echo "::group::Testing migration reversibility"
# Get current revision
CURRENT=$(uv run alembic current 2>&1 | grep -oE "^[a-f0-9]+" | head -1)
CURRENT=$(uv run --frozen alembic current 2>&1 | grep -oE "^[a-f0-9]+" | head -1)

if [ -z "$CURRENT" ]; then
echo "No migrations applied, skipping cycle test"
Expand All @@ -108,14 +108,14 @@ jobs:

# Downgrade one step
echo "Downgrading one migration..."
uv run alembic downgrade -1
uv run --frozen alembic downgrade -1

# Upgrade back
echo "Upgrading back to head..."
uv run alembic upgrade head
uv run --frozen alembic upgrade head

# Verify we're back at head
FINAL=$(uv run alembic current 2>&1 | grep -oE "^[a-f0-9]+" | head -1)
FINAL=$(uv run --frozen alembic current 2>&1 | grep -oE "^[a-f0-9]+" | head -1)

if [ "$CURRENT" != "$FINAL" ]; then
echo "::error::Migration cycle failed - revision mismatch after downgrade/upgrade"
Expand All @@ -133,10 +133,10 @@ jobs:
echo "" >> $GITHUB_STEP_SUMMARY
echo "### Migration History" >> $GITHUB_STEP_SUMMARY
echo '```' >> $GITHUB_STEP_SUMMARY
uv run alembic history --verbose 2>&1 | head -50 >> $GITHUB_STEP_SUMMARY
uv run --frozen alembic history --verbose 2>&1 | head -50 >> $GITHUB_STEP_SUMMARY
echo '```' >> $GITHUB_STEP_SUMMARY
echo "" >> $GITHUB_STEP_SUMMARY
echo "### Current State" >> $GITHUB_STEP_SUMMARY
echo '```' >> $GITHUB_STEP_SUMMARY
uv run alembic current --verbose 2>&1 >> $GITHUB_STEP_SUMMARY
uv run --frozen alembic current --verbose 2>&1 >> $GITHUB_STEP_SUMMARY
echo '```' >> $GITHUB_STEP_SUMMARY