-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
feat(core): allow loading env vars by target and configuration #14998
feat(core): allow loading env vars by target and configuration #14998
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
856a77f
to
bf5f755
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Double Triple check the order in the code and the docs.
bf5f755
to
9515f2c
Compare
9515f2c
to
3427e10
Compare
Great PR. Thanks. Are the docs gonna be updated as well? |
The docs updates are in this PR |
Ow.. You're right. Thought it where the tests. My bad. Thanks again 👍 |
(cherry picked from commit 24e87e6)
@AgentEnder Shouldn't we also load This is a very common requirement |
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. |
Current Behavior
Nx can load env vars by target name
Expected Behavior
Nx can load env vars by target name combined with configuration
Related Issue(s)
Fixes #14931