Skip to content

feat: allow dotenv-vars to be configured when deploying#60

Merged
Gandhi11 merged 1 commit intomainfrom
feature/dotenv-vars
Jul 29, 2025
Merged

feat: allow dotenv-vars to be configured when deploying#60
Gandhi11 merged 1 commit intomainfrom
feature/dotenv-vars

Conversation

@xel1045
Copy link
Copy Markdown
Member

@xel1045 xel1045 commented Jun 19, 2025

This pull request introduces a new deployment task to handle .env variable updates during the deployment process in the Envoy.blade.php file. The most important changes include defining and processing .env variables, adding a new deployment task for .env updates, and integrating this task into existing deployment flows.

New .env Variable Handling:

  • Added logic to parse and filter .env variables from the $dotenvVars string into an array for further processing. (Envoy.blade.php, Envoy.blade.phpR7-R8)

New Deployment Task:

  • Introduced a new deploy:dotenv task to update .env files during deployment. This task iterates over the parsed .env variables, processes them, and updates the .env file using the sed command. (Envoy.blade.php, Envoy.blade.phpR261-R275)

Integration into Deployment Flow:

  • Integrated the deploy:dotenv task into the deployment sequence by adding it to the task lists for deploy and deploy:provisioned. (Envoy.blade.php, [1] [2]

@xel1045 xel1045 requested a review from Gandhi11 June 19, 2025 16:33
@xel1045 xel1045 self-assigned this Jun 19, 2025
@xel1045 xel1045 force-pushed the feature/dotenv-vars branch from 2d02dd8 to bb91db9 Compare June 19, 2025 16:37
@Gandhi11 Gandhi11 force-pushed the feature/dotenv-vars branch 9 times, most recently from aea92c2 to e883622 Compare July 1, 2025 20:11
@Gandhi11 Gandhi11 force-pushed the feature/dotenv-vars branch from e883622 to 74328f0 Compare July 29, 2025 13:14
@Gandhi11 Gandhi11 merged commit 918765a into main Jul 29, 2025
12 checks passed
@Gandhi11 Gandhi11 deleted the feature/dotenv-vars branch July 29, 2025 14:23
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