Skip to content

Remove hub requirement from pull-requests documentation #586

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.

Sign up for GitHub

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

Jump to bottom
Merged
swissspidy merged 4 commits into from
Dec 11, 2025
Merged

Remove hub requirement from pull-requests documentation #586

swissspidy merged 4 commits into from
Dec 11, 2025

Conversation

Copy link
Contributor

Copilot AI commented Nov 2, 2025 •
edited
Loading

Remove "hub" requirement from documentation

Changes Completed

  • Remove "hub" requirement and references from pull-requests.md
    • Remove hub from installation requirements
    • Replace hub clone with git clone command
    • Replace hub fork references with manual git instructions
    • Add optional mention of gh CLI as a helper tool
  • Add links to GitHub documentation for forking and adding remotes

Summary

All references to the hub tool have been removed. The documentation now:

  1. Uses only Composer as a required tool
  2. Uses standard git clone commands
  3. Provides manual instructions for forking via GitHub UI with links to official GitHub documentation
  4. Links to GitHub docs on managing remote repositories
  5. Mentions gh CLI as an optional helper

The changes make the documentation more accessible by using standard git commands while providing helpful links to GitHub's official documentation for tasks like forking and managing remotes.

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove "hub" requirement</issue_title>
<issue_description>## Feature Request

Currently, https://make.wordpress.org/cli/handbook/contributions/pull-requests/ indicates hub is a required tool

Install Composer and hub if you don’t already have them.

and some commands are hub specific, e.g.

hub clone wp-cli/core-command

Proposed Solution

Modify the language and example commands to use git commands directly, rather than hub.

Optionally, call out hub as a useful tool that can be used.

Possible Drawbacks

  1. Commands will be slightly more verbose.
  2. Will we need to differentiate between a HTTPS or SSH clone? 
    git clone [email protected]:wp-cli/wp-cli.git ~/wp-cli
```</issue_description>

<agent_instructions>Mention the gh CLI tool as an optional helper. It's not mandatory. Don't mention hub.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@danielbachhuber @salcode Did you want to submit a PR for this?

💡 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.

Copilot AI self-assigned this Nov 2, 2025
Copilot AI changed the title [WIP] Remove hub requirement from contribution documentation Remove hub requirement from pull-requests documentation Nov 2, 2025
Copilot AI requested a review from swissspidy November 2, 2025 17:26
@swissspidy swissspidy marked this pull request as ready for review November 2, 2025 18:13
@swissspidy swissspidy requested a review from a team as a code owner November 2, 2025 18:13
wojsmol
wojsmol reviewed Nov 2, 2025
View reviewed changes
@swissspidy
Copy link
Member

swissspidy commented Nov 3, 2025

@copilot For some of the parts of the documentation, such as forking a repository, link to relevant GitHub documentation.

Copy link
Contributor Author

Copilot AI commented Nov 3, 2025

@copilot For some of the parts of the documentation, such as forking a repository, link to relevant GitHub documentation.

Added links to GitHub documentation for forking repositories and managing remotes in commit 3122596.

@swissspidy swissspidy merged commit 7aa361f into main Dec 11, 2025
7 checks passed
@swissspidy swissspidy deleted the copilot/remove-hub-requirement branch December 11, 2025 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swissspidy swissspidy Awaiting requested review from swissspidy

1 more reviewer

@wojsmol wojsmol wojsmol left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@swissspidy swissspidy

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove "hub" requirement

3 participants

@swissspidy @wojsmol