Skip to content
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

fix(core): expand env variables on load and unload #26459

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

xiongemi
Copy link
Collaborator

@xiongemi xiongemi commented Jun 7, 2024

This pr is meant to replace #22585 and #20524

Env variables using other variables were not unloaded from the environment and further customizations were impossible in more specific env files.

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #23581

Copy link

vercel bot commented Jun 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jun 25, 2024 7:53pm

@xiongemi xiongemi marked this pull request as ready for review June 23, 2024 23:20
@xiongemi xiongemi requested a review from a team as a code owner June 23, 2024 23:20
@xiongemi xiongemi requested a review from AgentEnder June 23, 2024 23:20
@xiongemi xiongemi force-pushed the fix/nx-task-env branch 2 times, most recently from 1130c5f to 06d88a3 Compare June 25, 2024 01:35
@xiongemi xiongemi force-pushed the fix/nx-task-env branch 2 times, most recently from e9f50c3 to 326ae0c Compare June 25, 2024 02:56
@xiongemi xiongemi force-pushed the fix/nx-task-env branch 3 times, most recently from f65c3fe to e58880f Compare June 25, 2024 05:16
@xiongemi xiongemi force-pushed the fix/nx-task-env branch 2 times, most recently from 7f62d19 to d84ae63 Compare June 25, 2024 17:08
packages/nx/src/tasks-runner/task-env.ts Outdated Show resolved Hide resolved
packages/nx/src/tasks-runner/task-env.ts Outdated Show resolved Hide resolved
packages/nx/src/tasks-runner/task-env.ts Outdated Show resolved Hide resolved
Env variables using other variables were not unloaded from the environment
and further customizations were impossible in more specific env files.
@FrozenPandaz FrozenPandaz merged commit 88fd03b into master Jun 26, 2024
5 checks passed
@FrozenPandaz FrozenPandaz deleted the fix/nx-task-env branch June 26, 2024 14:19
FrozenPandaz pushed a commit that referenced this pull request Jun 26, 2024
This pr is meant to replace #22585 and
#20524

Env variables using other variables were not unloaded from the
environment and further customizations were impossible in more specific
env files.

<!-- Please make sure you have read the submission guidelines before
posting an PR -->
<!--
https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr
-->

<!-- Please make sure that your commit message follows our format -->
<!-- Example: `fix(nx): must begin with lowercase` -->

## Current Behavior
<!-- This is the behavior we have today -->

## Expected Behavior
<!-- This is the behavior we should expect with the changes in this PR
-->

## Related Issue(s)
<!-- Please link the issue being fixed so it gets closed when this is
merged. -->

Fixes #23581

Co-authored-by: Mateo Tibaquira <[email protected]>
(cherry picked from commit 88fd03b)
Copy link

github-actions bot commented Jul 2, 2024

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nx:run-commands executor ignore envFile option
3 participants