Skip to content

Feat: Test migrations and seeders in GitHub CI#517

Merged
rinkp merged 1 commit intoGEWIS:mainfrom
rinkp:feat/ci-migration-test
Feb 15, 2026
Merged

Feat: Test migrations and seeders in GitHub CI#517
rinkp merged 1 commit intoGEWIS:mainfrom
rinkp:feat/ci-migration-test

Conversation

@rinkp
Copy link
Copy Markdown
Member

@rinkp rinkp commented Feb 8, 2026

Description

This pull request introduces a simple check for migrations and seeders. Whenever seeders/migrations have been changed, it does the following checks:

  • Migrate base branch
  • Seed base branch
  • Test forward migration to head branch
  • Seed using seeders of head branch
  • Rollback migrations to base branch migrations

Related issues/external references

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation improvement (no changes to code)
  • Other: CI change

@rinkp rinkp requested a review from tomudding February 8, 2026 21:04
@rinkp
Copy link
Copy Markdown
Member Author

rinkp commented Feb 8, 2026

This is expected to fail now (because GH-516 needs to be merged to repair building AND because the docker-compose.github.yml file is not yet present on the base branch). See https://github.com/GEWIS/gewisdb/actions/runs/21805157073 for an example of a succesful run.

Copy link
Copy Markdown
Member

@tomudding tomudding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work!

@rinkp rinkp force-pushed the feat/ci-migration-test branch from 9cec64e to c0ce498 Compare February 15, 2026 18:37
@rinkp rinkp enabled auto-merge February 15, 2026 18:37
@rinkp rinkp merged commit ced0fa5 into GEWIS:main Feb 15, 2026
4 checks passed
@rinkp rinkp deleted the feat/ci-migration-test branch February 15, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants