Skip to content

#329 Add environment variable support for migrations directory#864

Merged
mfridman merged 3 commits intopressly:masterfrom
rwwwx:329-add-env-vars-support-for-migrations-directory
Dec 14, 2024
Merged

#329 Add environment variable support for migrations directory#864
mfridman merged 3 commits intopressly:masterfrom
rwwwx:329-add-env-vars-support-for-migrations-directory

Conversation

@rwwwx
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

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

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