-
Notifications
You must be signed in to change notification settings - Fork 357
Document ssh+path configuration limitation in wp-cli.yml #622
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
Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
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.
Pull request overview
This pull request documents a known limitation where the path configuration option is ignored when specified alongside ssh as separate top-level options in wp-cli.yml, though it works correctly when used in aliases or via command-line parameters.
Changes:
- Added "Using path with SSH" subsection explaining the limitation and providing three working approaches
- Documented that separate top-level
sshandpathoptions don't work together, but work fine in aliases
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| When using SSH connections, you can specify the remote WordPress path in two ways: | ||
|
|
||
| 1. **As a suffix in the SSH connection string** (recommended): | ||
| ``` |
Copilot
AI
Feb 3, 2026
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.
The first code block is missing the yaml language marker. The second code block at line 348 has yaml marker, but this one doesn't. For consistency, both should have the same format.
| ``` | |
| ```yaml |
| path: /path/to/wordpress | ||
| ``` | ||
|
|
||
| **Important:** When using `ssh` and `path` as separate top-level options in `wp-cli.yml` (not within an alias), the `path` option may not be applied correctly. In such cases, use one of the following workarounds: |
Copilot
AI
Feb 3, 2026
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.
The wording "may not be applied correctly" is vague. Based on the original issue description, the path option is actually ignored (not applied at all), not just incorrectly applied. Consider changing this to "the path option is not applied" or "the path option will be ignored" for more accurate documentation.
| **Important:** When using `ssh` and `path` as separate top-level options in `wp-cli.yml` (not within an alias), the `path` option may not be applied correctly. In such cases, use one of the following workarounds: | |
| **Important:** When using `ssh` and `path` as separate top-level options in `wp-cli.yml` (not within an alias), the `path` option will be ignored. In such cases, use one of the following workarounds: |
When
sshandpathare specified as separate top-level options inwp-cli.yml, thepathoption is not applied. This works correctly in aliases and on the command line, but not at the config root level.Changes
Added "Using path with SSH" subsection to Remote (SSH) configuration documenting:
sshandpathoptions don't work together (path is ignored)ssh: user@host~/path/to/wordpress@alias: { ssh: user@host, path: /path }wp --path=/path commandExample
Instead of:
Use:
Or define an alias:
Original prompt
pathwp-cli#5324💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.