Skip to content

Update README to provide a better explanation of setting up#56

Merged
danielbachhuber merged 2 commits intomainfrom
installation-development-updates
Nov 9, 2023
Merged

Update README to provide a better explanation of setting up#56
danielbachhuber merged 2 commits intomainfrom
installation-development-updates

Conversation

@danielbachhuber
Copy link
Copy Markdown
Member

@danielbachhuber danielbachhuber added the scope:documentation Related to documentation label Nov 4, 2023
@danielbachhuber danielbachhuber added this to the 2.0.1 milestone Nov 4, 2023
@danielbachhuber danielbachhuber self-assigned this Nov 4, 2023
@danielbachhuber danielbachhuber requested a review from a team as a code owner November 4, 2023 14:43
2. Install all Composer dependencies for a complete `wp-cli-bundle` setup, while symlinking all of the previously cloned packages into the Composer `vendor` folder.
3. Symlink all folder in `vendor` into corresponding `vendor` folders in each repository, thus making the centralized functionality based on Composer available in each repository subfolder.

First, clone the GitHub repository:
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the details you've added. I'm wondering if we could restructure it to keep only one list of steps. When I read the current version, after reading the first bullet I open the terminal and start thinking about how to clone all packages from the organization. It's not fully clear if I should perform steps 1-3 and then follow the steps below, or if the steps below are detailed instructions for steps 1-3. If the latter is true, could we compile them in one list that explains what should be done in a given step and include a sample command?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough, I consolidated to a single code block with 056411e

git clone https://github.com/wp-cli/wp-cli-dev wp-cli-dev
```

Before you can proceed further, you'll need to make sure you have [Composer](https://getcomposer.org/), PHP, and a functioning MySQL or MariaDB server on your local machine too.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to mention experimental SQLite support here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@swissspidy I'm open to it! What language would you suggest?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe something along these lines:

Alternatively, if you do not have MySQL or MariaDB installed on your computer, you can also run tests using SQLite, which is an alternative database that does not require separate installation. To do so, set the WP_CLI_TEST_DBTYPE environment variable accordingly when running tests. Example: WP_CLI_TEST_DBTYPE=sqlite composer behat
Note: In this case you do not need to run composer prepare-tests either.

Needs some pointer to the Behat docs I suppose.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add these in a moment.

@danielbachhuber danielbachhuber merged commit 2ef49b3 into main Nov 9, 2023
@danielbachhuber danielbachhuber deleted the installation-development-updates branch November 9, 2023 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:documentation Related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants