Skip to content
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

Move code to main branch (finally!) #248

Merged
merged 353 commits into from
Oct 27, 2022
Merged

Move code to main branch (finally!) #248

merged 353 commits into from
Oct 27, 2022

Conversation

ConradBoydElliottGustafson
Copy link
Contributor

No description provided.

* Refactoring: git mv from ./backend to ./server/backend

* Tweak for vscode pytest discovery.

* alembic to flyway migrations

* renamed the alembic migrations to match the names of equivalent migrations in flway dir
* altered the alembic behavious so instead of a hash as a revision id for the
migration it names it V1.. V2.. to correspond with how flyway names its migrations
* altered the alembic.ini so it uses the new migration name when generating new migrations
* started creation of script to automate the generation of flyway migrations

* update readme

* Adding seed data to alembic migration

* figure out the pattern for adding seed data using alembic
* added the seed data in the V2 flyway migration to alembic

* completed alembic / flway sync

* 1:1 relationship exists between alembic migrations and flyway
* script created to check / generate flyway migrations from alembic.
* minor tweaks to flake8 config after moving backend to server/backend
* started documentation... next - cleanup of docs for migrations

* completed the documentation of alembic steps

* update to vscode paths

after moving the ./backend directory to ./server/backend needed to patch
up some of the paths

Co-authored-by: Ian Liu <[email protected]>
@ConradBoydElliottGustafson
Copy link
Contributor Author

Merging to main branch to make github actions work!

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.

4 participants