-
-
Notifications
You must be signed in to change notification settings - Fork 163
Closed
Labels
Description
Hey!
So this may seem like a feature request, but it actually falls under the bug category in my opinion.
As of today, umzug is written and built in CJS.
The problem is, this breaks applications written in pure ESM because umzug is using a dynamic require.
This problem is currently affecting several ORMs (including sequelize).
I think migrating to ESM-first is necessity since ESM supports importing any kind of module unlike CJS.
I am willing to write a PR to that effect if reviewers are available to check it.
This will obviously create many changes within this dependency. It should not, however, create any breaking change.
Let me know if you are willing to migrate and if you want a PR implementing this change.