Skip to content

Conversation

@rragundez
Copy link
Contributor

@rragundez rragundez commented Nov 19, 2025

IMPORTANT: PR #223 has to be merged first

At the moment the .env file is sourced from inside the src directory, it should not be part of the code is best practice to make it part of the top level of the project. This PR makes the changes such that all scripts respect that, including the sourcing of the file from the source code.
There are a couple of benefits to doing this, like better visibility, separation of concerns between code functionality and dynamic configuration settings, and help the easy transition between using an .env file and environment variables set before runtime.

@rragundez rragundez changed the title Move source location of the .env file to the root at the project [WIP] Move source location of the .env file to the root at the project Nov 19, 2025
@rragundez rragundez mentioned this pull request Nov 19, 2025
@igorbenav igorbenav requested a review from LucasQR November 19, 2025 17:39
@rragundez rragundez changed the title [WIP] Move source location of the .env file to the root at the project Move source location of the .env file to the root at the project Nov 20, 2025
@rragundez
Copy link
Contributor Author

@LucasQR FYI once #223 is merged I would have to rebase and force pushed based on the squashed PR commit. So it might appear with repeated change as in 223 in the meantime.

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.

1 participant