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

Upgrades dotenv and dotenv-exapand with necessary code changes #880

Closed
wants to merge 2 commits into from
Closed

Upgrades dotenv and dotenv-exapand with necessary code changes #880

wants to merge 2 commits into from

Conversation

benkroeger
Copy link

@benkroeger benkroeger commented Feb 21, 2022

dotenv-expand changed their interface with 6.0.0.
This PR makes pg-migrate compatible with the old and new interfaces

closes #873
closes #876

checks for dotenv-expand's new interface first (introduced in 6.0.0)

closes #873
closes #876

checks for dotenv-expand's new interface first (introduced in 6.0.0 motdotla/dotenv-expand@14b1f28#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R31)

Signed-off-by: Benjamin Kroeger <[email protected]>
@JeromeBu
Copy link

Any clue why this does not get merged ?

@NigelGreenway
Copy link

Hey.. wondered what the status is on this PR? 😊

@JeromeBu JeromeBu mentioned this pull request Jun 24, 2022
@JeromeBu
Copy link

waiting for a review from a maintainer I guess... @littlewhywhat maybe ? 

@littlewhywhat
Copy link
Contributor

Hi, the fix is released in 6.2.2. I decided to drop support for dotenv-expand lower than 6.0.0. If problems persist or support for lower dotenv-expand is needed, let me know, I'll react asap. I apologise for making all of you wait for so long and thank you very much for being active and creating a fix.

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.

Error when trying to migrate with usage of "dotenv-expand"
7 participants