-
Notifications
You must be signed in to change notification settings - Fork 476
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
allow import <variable-name>
#1975
Comments
The same need for all set dotenv-path := "" + MPM_CONFIG_DIR + "/config" I want to create an application mainly using |
The inability to do variable (or environment variable) substitution for import paths ended up preventing me from a desired DRYing out of Justfile content as well, rather than having to duplicate a Justfile name that I want to include as part of replicating a subset of local contents. It appears that just's GRAMMAR (currently) accepts only strings for imports as well as modules; so not sure if there is any easy way to implement this without big impacts. |
I can see why this is desirable, but it's tough at the implementation level. Justfiles are statically analyzed before anything runs, including any expression evaluation. I would like to support this, but not at the cost of making the implementation much more complicated. #1849 partially fixes this, since it allows for expanding environment variables inside of import paths, although it doesn't support arbitrary expressions, and environment variables are loaded without taking into account things like |
For example:
The text was updated successfully, but these errors were encountered: