-
Notifications
You must be signed in to change notification settings - Fork 121
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
Make possible to provide the env file pathes in the Makefile.toml
#306
Comments
nice, few questions:
env-files = [
"./myfile.env",
"${HOME}/anotherfile.env",
{ file = "dev1.env", profile = "dev" },
{ file = "dev2.env", profile = "dev" },
{ file = "test.env", profile = "test" },
] few things about this example:
would this fit your needs? |
Your solution with profiles in parentheses is good but maybe it would be better to make it like For example following code:
will work same as:
with corresponding env files. And if somebody will want to add global env file something like this:
|
just that i'm clear on your solution, basically you are saying to expand the capabilities of if i understood correctly, the problem here [env]
env-files = [
"./dev1.env",
"${HOME}/dev2.env",
] is that you are blocking env-files from being an env var key without the ability to override that behaviour. i'm open for suggestions on how to make it simpler. as for the task example you gave:
it is what i meant in my original post:
so i think we are fine with that. |
@olvyko and @enfipy this feature is now available in the 0.24.0 development branch. |
this is no published officially in 0.24.0 release. |
Features Description
Make possible to provide the env file path (
--env-file
) in theMakefile.toml
.Describe the solution you'd like
Now, when you use
--env-file
, you need to callcargo make
like this:but it would be cool if you could provide the env file pathes in the
Makefile.toml
,Code Sample
(for example, but can be implemented differently)
then just run
cargo make dev
orcargo make test
and allenv-files
will be attached automatically.The text was updated successfully, but these errors were encountered: