-
Couldn't load subscription status.
- Fork 87
[Docs]: Add comprehensive environment variables documentation #602
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
base: main
Are you sure you want to change the base?
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
|||||||||
|
|
||
| In addition to the project compose files, an optional Preevy-specific Compose file can be used. Preevy attempts to load files named `compose.preevy.yaml`, `compose.preevy.yml`, `docker-compose.preevy.yaml` or `docker-compose.preevy.yml`. If one of these exists, it is loaded BEFORE the project composes file(s). The name of the Preevy-specific compose file can be overridden by specifying the argument `--system-compose-file`. | ||
|
|
||
| ### Environment Variables |
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.
Try to minimize it and a link to recpies example you've created (and docker compose relevant documentation)
| DESCRIPTION | ||
| Bring up a preview environment | ||
| ENVIRONMENT VARIABLES |
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.
This file is generated, so no need to add it here
|
|
||
| ### Environment variables not being interpolated | ||
|
|
||
| If environment variables aren't being substituted, check: |
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.
No need to be strongly opinionated about the practices, there's docker compose documentation for that, try to emphasis simple common simple cases, preevy is used for preview environments not production environments.
User description
Addresses the recurring issue where users are unaware that Preevy fully supports Docker Compose environment variable features. This PR adds comprehensive documentation to help users understand how to pass environment variables to their preview environments.
Problem
As noted in issue #XXX and confirmed by @Yshayy, users frequently ask about passing environment variables to
preevy up, not realizing that Preevy already supports all Docker Compose environment variable capabilities. This is purely a documentation gap rather than a missing feature.Solution
Added comprehensive documentation across multiple locations:
1. Enhanced README.md
Added a new "Environment Variables" section in the Configuration chapter that covers:
.envfiles for configuration management2. Updated CLI Documentation
Enhanced the
preevy upcommand documentation with:.envfiles3. Created Comprehensive Recipe Guide
Added
site/docs/recipes/environment-variables.mdcontaining:Examples Added
The documentation now clearly shows users how to use environment variables in multiple ways:
And how to pass them from CI:
Impact
This documentation eliminates the confusion around environment variable support and provides clear guidance for:
No code changes were required as Preevy already supports all Docker Compose environment variable capabilities through its underlying Docker Compose integration.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
PR Type
Documentation
Description
Add comprehensive environment variables documentation across multiple locations
Enhance README.md with new Environment Variables section
Update CLI documentation with practical examples
Create detailed recipe guide for environment variable usage
Diagram Walkthrough
File Walkthrough
README.md
Add Environment Variables configuration sectionREADME.md
up.md
Enhance up command with environment variables documentationpackages/cli/docs/up.md
up.md
Update CLI reference with environment variables sectionsite/docs/cli-reference/up.md
environment-variables.md
Create complete environment variables recipe guidesite/docs/recipes/environment-variables.md