Open codebymikey opened 3 months ago
Open the branch in Web Editor • VS Code • Insiders
Open Preview
The changes involve modifying several shell script templates within the .gitpod/utils/script-templates/
directory. Specifically, the output of echo
commands in these scripts is now redirected to stderr
by adding >&2
before each echo
statement. This ensures that messages are sent to the error stream, which can be useful for logging and debugging purposes.
Files | Change Summary |
---|---|
.gitpod/utils/script-templates/ddev-composer.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
.gitpod/utils/script-templates/ddev-drush.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
.gitpod/utils/script-templates/ddev-node.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
.gitpod/utils/script-templates/ddev-npx.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
.gitpod/utils/script-templates/ddev-nvm.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
.gitpod/utils/script-templates/ddev-php.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
.gitpod/utils/script-templates/ddev-yarn.template.sh |
Redirect echo output to stderr by adding >&2 before echo . |
In the land of code, where scripts reside,
Echoes now to stderr, they confide.
For debugging dreams and logs so clear,
These whispers to the error stream we steer.
A rabbit's touch, so gentle and wise,
Enhances our scripts, beneath coder skies.
🌟🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
The Problem/Issue/Bug
When trying to run certain scripts, the warning added to the output is added directly to the standard output, and messes up the expected output.
How this PR Solves The Problem
It adds the notice to the standard output stream instead.
Ideally this notice should be behind an environment variable.
Manual Testing Instructions
Related Issue Link(s)
Release/Deployment notes
Summary by CodeRabbit