Skip to content

Conversation

sd-yip
Copy link

@sd-yip sd-yip commented Feb 4, 2021

Previously, the command would overwrite the HOST and PORT variables based on the hostname and port of the service which we would check for being available. This change prevents these variables from being overwritten, which allows you to set them outside the command.

BREAKING CHANGE: HOST, PORT and other internally used environment variables are not overwritten anymore. If you use these, then you need to manually supply them.

Previously, the command would overwrite the HOST and PORT variables based on the hostname and port of the service which we would check for being available. This change prevents these variables from being overwritten, which allows you to set them outside the command.

BREAKING CHANGE: HOST, PORT and other internally used environment variables are not overwritten anymore. If you use these, then you need to manually supply them.
@sd-yip
Copy link
Author

sd-yip commented Feb 4, 2021

This pull request contains the content of 9964904 that was previously in #27.

@Addono I have added 2 test cases to the commit additionally. Please take a look.

@Addono Addono merged commit c7631e5 into eficode:master Feb 4, 2021
@github-actions
Copy link

github-actions bot commented Feb 4, 2021

🎉 This PR is included in version 2.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@sd-yip sd-yip deleted the fix/environment-restoration branch February 4, 2021 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants