Skip to content

Comments

temp: Add dummy model for testing migrations [BOMS-227]#1138

Merged
timmc-edx merged 2 commits intomainfrom
timmc/test-migrations
Nov 17, 2025
Merged

temp: Add dummy model for testing migrations [BOMS-227]#1138
timmc-edx merged 2 commits intomainfrom
timmc/test-migrations

Conversation

@timmc-edx
Copy link
Member

Also includes a commit to clean up the changelog headings.

Output of sqlmigrate edx_arch_experiments 0001_initial:

CREATE TABLE "edx_arch_experiments_boms227" ("id" integer NOT NULL PRIMARY KEY AUTOINCREMENT, "dummy_field" varchar(50) NOT NULL);

With --backwards:

DROP TABLE "edx_arch_experiments_boms227";

Merge checklist:
Check off if complete or not applicable:

  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Fixup commits are squashed away
  • Unit tests added/updated
  • Manual testing instructions provided
  • Noted any: Concerns, dependencies, migration issues, deadlines, tickets

Copy link
Contributor

@robrap robrap left a comment

Choose a reason for hiding this comment

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

Later, your plan is not to revert the migration (from code), but to revert the model and and create a new migration for the delete?

@timmc-edx
Copy link
Member Author

Correct -- adding a second migration.

Use heading bars that are consistent internally and with the rest of the
codebase.
This will be followed later by a revert of the model and addition of a
corresponding second migration.

Output of `sqlmigrate edx_arch_experiments 0001_initial`:

```
CREATE TABLE "edx_arch_experiments_boms227" ("id" integer NOT NULL PRIMARY KEY AUTOINCREMENT, "dummy_field" varchar(50) NOT NULL);
```

With `--backwards`:

```
DROP TABLE "edx_arch_experiments_boms227";
```

BOMS-227
@timmc-edx timmc-edx force-pushed the timmc/test-migrations branch from a38da62 to 5f0351e Compare November 17, 2025 16:50
@timmc-edx timmc-edx merged commit 9eed984 into main Nov 17, 2025
6 checks passed
@timmc-edx timmc-edx deleted the timmc/test-migrations branch November 17, 2025 16:53
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