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

Replace migration mechanism #2654

Merged
merged 3 commits into from
Oct 3, 2022
Merged

Replace migration mechanism #2654

merged 3 commits into from
Oct 3, 2022

Conversation

taylortom
Copy link
Member

@taylortom taylortom commented Sep 8, 2022

Fixes #2653

Unfortunately I've had to rip out the old migration mechanism due to incompatibility issues. I've added a new npm task (create-migration) to set up new migrations, everything else is run internally, so nothing changes.

All existing migrations have been updated to use the new module (and renamed for consistency).

@taylortom taylortom added this to the 0.11.0 THE FINAL LEGACY milestone Sep 9, 2022
@deltanetdan
Copy link

@taylortom if I install using this branch (issue/2653)) I get the following error after "Web application built successfully" log:

ERROR:  Error: Migrator#runTask expects a callback parameter
    at runMigrations (C:\Users\dan.gray\Documents\Source\Adapt\local\adapt-test-final\at-migrate\lib\installHelpers.js:317:11)

Tried with node v14.19.3 and v16.15.1.

@taylortom taylortom merged commit 40c6176 into release/v0.11.0 Oct 3, 2022
@taylortom taylortom deleted the issue/2653 branch October 3, 2022 14:23
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