Errors installing command into development environment about package-command

[danielbachhuber]

Thanks for the report, @Roy-Orbison !

We can definitely try to fix up the URL syntaxes you think are broken. I'm having a little bit of a hard time grokking your report, though. Which syntaxes would you like to see supported?

[Roy-Orbison]

In this docs section, it states that adding a module is done like so, the last parameter looking like a typical Git URL:

wp package install [email protected]:wp-cli/dist-archive-command.git

These all failed:

• https://github.com/wp-cli/search-replace-command.git
• github:wp-cli/search-replace-command.git (where github: is an SSH alias that sets the user and selects the SSH key)
• github:wp-cli/search-replace-command.git:dev-main (trying to select correct branch)
• https://github.com/wp-cli/search-replace-command.git:dev-main

What worked was wp-cli/search-replace-command:dev-main, which I gleaned from this issue comment where someone else had encountered the invalid default branch dev-master.

You don't necessarily have to support those URL types, more that the docs didn't match the experience, so maybe something needs to be updated.

[danielbachhuber]

Thanks, @Roy-Orbison.

I think we should:

• Support additional URL patterns where doing so wouldn't cause a backcompat break.
• Document the URL patterns we don't support because they conflict with some existing pattern.

In honestly, what's most likely is that the additional URL patterns weren't considered when the feature was created.

Feel free to submit a pull request, if you'd like. Here is some guidance on our pull request best practices.