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

💡 Add documentation for migration generation / drivers #492

Open
theilm opened this issue Jul 26, 2024 · 1 comment
Open

💡 Add documentation for migration generation / drivers #492

theilm opened this issue Jul 26, 2024 · 1 comment
Labels
type:feature New feature.

Comments

@theilm
Copy link

theilm commented Jul 26, 2024

When generating migrations, the generated migration code depends on the driver that is currently used. This might lead to problems when using the migration with another driver later on.

Specifically, we encountered a problem when we generated migrations with MySQL driver. The "size" property was added to primary / integer columns. This lead to problems when executing the migration with SQLite driver. When removing the "size" property, the migration worked fine with SQLite.

Should we add documentation regarding this behavior? Should we add suggestions what driver to use for generating migrations in order to ensure compatibility with most drivers / which properties to remove?

@theilm theilm added the type:feature New feature. label Jul 26, 2024
@roxblnfk roxblnfk moved this to Todo in Cycle Jul 26, 2024
@wolfy-j
Copy link
Contributor

wolfy-j commented Jul 28, 2024

Any PRs will be welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New feature.
Projects
Status: Todo
Development

No branches or pull requests

2 participants