Fix: set PWD only if not set by kianelbo · Pull Request #13268 · docker/compose
Conversation
What I did
Check whether a PWD env variable is present and only if it isn't, set it to the result of os.Getwd().
Related issue
Fixes #13265
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT with a minor naming issue
| pwd, err := os.Getwd() | ||
| if err != nil { | ||
| return nil, err | ||
| envOptions := []cli.ProjectOptionsFn{ |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
envOptions is a bad name, as those many options are not only set to manage env, but many aspected of project setup. A basic opts would be better
Signed-off-by: Kian Eliasi <kian.elbo@gmail.com>
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request
Oct 20, 2025This 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
2 participants