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

Breaking change in v3.3.0 #614

Closed
ddolcimascolo opened this issue Aug 2, 2023 · 5 comments
Closed

Breaking change in v3.3.0 #614

ddolcimascolo opened this issue Aug 2, 2023 · 5 comments

Comments

@ddolcimascolo
Copy link

Hi guys,

Umzug v3.3.0 introduces a breaking change, there's now an id column expected in the migrations table when using sequelize storage. Running migrations now fail because this column does not exist in the database...

Can you revert this and release a 3.3.1 as soon as possible?

Thx,
David

@ddolcimascolo
Copy link
Author

@mmkal @GuyEphraim

I'm bringing your attention on this issue. Sorry I don't have much details handy as I'm not at work and only have CI logs, but the problem is pretty obvious by looking at 15fc2fe

Anything I can do to help?

Thx,
David

@wangshijun
Copy link

Same issue here

@mmkal
Copy link
Contributor

mmkal commented Aug 2, 2023

Thanks for the report. I will try to publish a revert in the next day or so. In the meantime, please use 3.2.1 (or ~3.2.1, but not ^3.2.1).

mmkal added a commit that referenced this issue Aug 3, 2023
@mmkal
Copy link
Contributor

mmkal commented Aug 3, 2023

3.3.1 released

3.3.0 deprecated (github release edited)

@mau31415 @GuyEphraim if you would like to un-revert/open a new PR please feel free. In the meantime I may try to write a breaking-change-smoke-test for SequelizeStorage.

@mmkal mmkal closed this as completed Aug 3, 2023
@ddolcimascolo
Copy link
Author

Thx for fixing

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

No branches or pull requests

3 participants