docs: add a "deep dive" on environment variables#43243
Merged
Merged
Conversation
secustor
reviewed
May 11, 2026
felipecrs
reviewed
May 12, 2026
RahulGautamSingh
suggested changes
May 12, 2026
jamietanna
commented
May 12, 2026
jamietanna
force-pushed
the
docs/env-var-deep-div
branch
from
b0ad649 to
7af078e
Compare
jamietanna
commented
May 18, 2026
Contributor
Author
|
viceice
reviewed
May 26, 2026
viceice
left a comment
viceice
left a comment
Member
There was a problem hiding this comment.
LGTM, can't see the diagrams on mobile 😅
jamietanna
force-pushed
the
docs/env-var-deep-div
branch
2 times, most recently
from
9b5aaee to
5c65357
Compare
secustor
reviewed
May 29, 2026
This was referenced May 29, 2026
As part of recent work around environment variable handling, it's clear we should provide more user-facing documentation on how environment variables are handled, and which are passed to child processes by default. We can create a "deep dive" page for this, with an autogenerated list of the environment variables in `basicEnvVars` that are always passed to child processes, if they're set. We can also point self-hosted administrators and users alike to the options they have to control this, too. With help from GPT-4.1 for case-insensitive sorting. Co-authored-by: Claude Sonnet 4.6 <jamie.tanna+claude-code@mend.io> Co-authored-by: gpt-4.1 <jamie.tanna+github-copilot@mend.io>
jamietanna
force-pushed
the
docs/env-var-deep-div
branch
from
5c65357 to
45d6267
Compare
Contributor
Author
|
OK, rebased + made a few more amends per Sebastian's comments - otherwise good for a re-review and good to go! |
secustor
approved these changes
May 29, 2026
Contributor
|
🎉 This PR is included in version 43.204.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
As part of recent work around environment variable handling, it's clear
we should provide more user-facing documentation on how environment
variables are handled, and which are passed to child processes by
default.
We can create a "deep dive" page for this, with an autogenerated list of
the environment variables in
basicEnvVarsthat are always passed tochild processes, if they're set.
We can also point self-hosted administrators and users alike to the
options they have to control this, too.
Context
Please select one of the below:
AI assistance disclosure
Did you use AI tools to create any part of this pull request?
Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via:
The public repository: