Skip to content

Fix/issue 2362 remove create all#2369

Open
Prateek2007-cmd wants to merge 8 commits into
janavipandole:mainfrom
Prateek2007-cmd:fix/issue-2362-remove-create-all
Open

Fix/issue 2362 remove create all#2369
Prateek2007-cmd wants to merge 8 commits into
janavipandole:mainfrom
Prateek2007-cmd:fix/issue-2362-remove-create-all

Conversation

@Prateek2007-cmd

Copy link
Copy Markdown
Contributor

Description

Resolves Issue #2362 (Direct Schema Creation Over Migrations).

The backend application was previously reliant on models.Base.metadata.create_all() upon application startup. This initialization method is strictly intended for rapid prototyping and local development; it lacks any mechanism for version control, rollback capabilities, or schema evolution (e.g., executing ALTER TABLE). Deploying this to production is a severe liability that invariably leads to data corruption or data loss during deployment upgrades.

This PR permanently eradicates the create_all invocation from the application lifecycle to force compliance with a robust migration pipeline.

Changes Made

  • Technical Debt Purged: Removed create_all from backend/app/main.py.
  • Architectural Shift: Application startup is now completely decoupled from database schema modification. The repository now mandates the use of an external, stateful migration tool like Alembic to track and execute schema changes deterministically.

Type of Change

  • Architectural Hardening
  • Technical Debt Resolution

@vercel

vercel Bot commented Jun 20, 2026

Copy link
Copy Markdown

@Prateek2007-cmd is attempting to deploy a commit to the janavipandole's projects Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant