Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#329 Add environment variable support for migrations directory #864

Conversation

rwwwx
Copy link
Contributor

@rwwwx rwwwx commented Nov 28, 2024

added GoDotEnv to extract env vars from .env files #329

.env.example Outdated
@@ -0,0 +1,3 @@
GOOSE_DRIVER=postgres
Copy link
Collaborator

Choose a reason for hiding this comment

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

Mind removing this file, don't think we need to include it and probably sufficient to document it either in the README or docs site https://pressly.github.io/goose/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

extended README.md with this example

@rwwwx rwwwx requested a review from mfridman December 14, 2024 13:30
@mfridman
Copy link
Collaborator

Thanks for the PR. Given the popularity of the linked issue #329 seems reasonable to add.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants