-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Bugfix dotenv test in laravel.php #4082
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
Conversation
The test to check if .env file is present was not checking if the dotenv variable was set and always used the default. Added getter for dotenv variable.
@antonmedv as discussed in #4081 I created a pr for the bug in the laravel recipe for checking the .env file's presence. |
I think we also need to change dotenv itself, so it works during deploy: -set('dotenv', '{{current_path}}/.env');
+set('dotenv', '{{release_or_current_path}}/.env'); |
@antonmedv where should this be added? I cannot find any occurrence of the |
@antonmedv I'm happy to help but could you point me in the right direction? As stated in my previous comment I cannot find the line of code you shared besides a comment.
|
Change instructions for set dotenv to use release_or_current_path variable.
@antonmedv just added the commit. |
Change instructions for set dotenv to use release_or_current_path variable.
The test to check if .env file is present was not checking if the dotenv variable was set and always used the default. Added getter for dotenv variable.