Skip to content

Core: Correctly resolve injectEnvironmentFiles default value#7063

Open
clemyan wants to merge 3 commits intomasterfrom
clemyan/config-env-inject-order
Open

Core: Correctly resolve injectEnvironmentFiles default value#7063
clemyan wants to merge 3 commits intomasterfrom
clemyan/config-env-inject-order

Conversation

@clemyan
Copy link
Copy Markdown
Member

@clemyan clemyan commented Mar 9, 2026

What's the problem this PR addresses?

In Configuration.find, the default value of injectEnvironmentFiles ([".env.yarn?"]) is resolved before projectCwd, causing it to anchor to startingCwd instead.

Fixes #6111

How did you fix it?

Move resolving projectCwd to before resolving default values of any config

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@clemyan clemyan changed the title Clemyan/config env inject order Core: correctly resolve injectEnvironmentFiles default value Mar 11, 2026
@clemyan clemyan changed the title Core: correctly resolve injectEnvironmentFiles default value Core: Correctly resolve injectEnvironmentFiles default value Mar 11, 2026
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.

[Bug?]: .env.yarn is not loaded automatically when running yarn from a workspace directory

1 participant