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

Use MIGRATION_DIR environment variable #2419

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Conversation

sjunepark
Copy link
Contributor

I'm not sure whether this was intended or not, but the current cli command does not use a environment variable for the migrate command.

In my case using a MIGRATION_DIR was more easier to use, as I had to pass -d for every command I run if I'm not using the default directory.

If this seems unnecessary or misleading, just close the PR thanks.

Copy link
Member

@tyt2y3 tyt2y3 left a comment

Choose a reason for hiding this comment

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

thank you, it's indeed a nice addition!

@tyt2y3 tyt2y3 merged commit 3512c1f into SeaQL:master Dec 23, 2024
Copy link

🎉 Released In 1.1.3 🎉

Thank you everyone for the contribution!
This feature is now available in the latest release. Now is a good time to upgrade!
Your participation is what makes us unique; your adoption is what drives us forward.
You can support SeaQL 🌊 by starring our repos, sharing our libraries and becoming a sponsor ⭐.

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