You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[CLI] Make messaging clearer for errors and Xdebug IDE integration (#2869)
## Motivation for the change, related issues
This PR tweaks a couple of strings around Playground CLI messaging
noticed by @fellyph and I while discussing Xdebug IDE integration
documentation this afternoon,.
## Implementation details
This PR:
- Makes the message for an error with a `cause` a bit clearer by placing
a ":" after "caused by".
- Before: `Could not configure Xdebug caused by VS Code IDE integration
requested, but no '.vscode' directory was found in the current working
directory.` (reads a bit strange but seems like one message)
- After: `Could not configure Xdebug caused by: VS Code IDE integration
requested, but no '.vscode' directory was found in the current working
directory.` (with the added colon, it is clearer we are talking about
separate messages)
- Adds the following lines to the VSCode IDE integration instructions:
- "1. Ensure you have installed an IDE extension for PHP Debugging"
- "(The **PHP Debug** extension by **Xdebug** has been a solid option)"
## Testing Instructions (or ideally a Blueprint)
- CI
0 commit comments