Document config-shaping env vars delivered via loadEnv .env files#564
Draft
kylebernhardy wants to merge 2 commits into
Draft
Document config-shaping env vars delivered via loadEnv .env files#564kylebernhardy wants to merge 2 commits into
kylebernhardy wants to merge 2 commits into
Conversation
Companion to HarperFast/harper#1580 (fixes HarperFast/harper#1513): HARPER_CONFIG / HARPER_SET_CONFIG / HARPER_DEFAULT_CONFIG delivered via a component .env are now applied before config composition. Documents precedence, encrypted-value and componentsRoot limitations. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request adds a new section to the environment variables documentation explaining how config-shaping variables are loaded early from .env files starting in version 5.2.0. The review feedback suggests a minor clarification to specify that the override: true property belongs to the loadEnv configuration rather than a generic component.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🚀 Preview DeploymentYour preview deployment is ready! 🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-564 This preview will update automatically when you push new commits. |
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
2 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.
Companion docs for HarperFast/harper#1580 (fixes HarperFast/harper#1513):
HARPER_CONFIG/HARPER_SET_CONFIG/HARPER_DEFAULT_CONFIGdelivered via a component'sloadEnv.envfile are now applied before the root config is composed, making.envdelivery equivalent to a real process env var.Adds a Config-Shaping Variables section to the environment-variables reference: behavior, precedence (process env wins unless
override: true), and the limitations (encrypted values can't shape config;componentsRootcan't be redirected via.env; only the three vars are applied early).Note for review
Version-gated
<VersionBadge type="changed" version="v5.2.0" />on the assumption the core PR ships in the next minor frommain. Hold merging until harper#1580 lands and confirm the version then.Generated by an LLM (Claude Fable 5).