php-cli-tools

php-cli-tools copied to clipboard

Restore CI on wp-cli/php-cli-tools

1

**Done is:** * The unit test suite runs for every pull request.

danielbachhuber

Broken table layout when reporting WordPress vulnerabilities.

4

Hi, Running wp vuln status through WordPress 4.9.6 breaks the table layout when the package reports any vulnerabilities with WordPress version (not plugins, not themes. See screenshots). ### Expected behavior:...

oscarssanchez

Using make_progress_bar multiple times in a WP_CLI command displays wrong expected total time after the first use. Repro: Consider the following cli command ``` class TestMultipleProgressBars { private function run($message)...

ctzkane

I was confused by `phpstan analyse` output: "Parameter #2 $default of function cli\prompt expects string, false given." Soon I found out that problem is in type hinting for this function...

valeriySeregin

Progress bar goes to new line under windows

4

The progressbar instead of updating the current row print a new row with the updated values. To fix this I've found that can be esily fixed by changing into Bar.php...

marcovalloni

sprintf(): Too few arguments

2

Can't really figure this one out. Sometimes (not often, but often enough to end up in my logs and irritate me) \cli\line() causes a warning: sprintf(): Too few arguments. This...

nahkampf

Table::getDisplayLines() should not output excessive lines when no rows

3

Fix #164.

slaFFik

Add ability to add row in a loop to existing already-rendered table

3

## Feature Request - [x] Yes, I reviewed the [contribution guidelines](https://make.wordpress.org/cli/handbook/contributing/). **Describe your use case and the problem you are facing** I want to "register" an empty table, and then...

slaFFik

Table::getDisplayLines() can output excessive lines

1

## Bug Report - [x] Yes, I reviewed the [contribution guidelines](https://make.wordpress.org/cli/handbook/contributing/). - [x] Yes, more specifically, I reviewed the guidelines on [how to write clear bug reports](https://make.wordpress.org/cli/handbook/bug-reports/). **Describe the current,...

slaFFik

Fix unexpected test failure in test_strwidth()

1

In https://github.com/wp-cli/php-cli-tools/pull/155, we marked a test skipped 217cb51 because it was unexpectedly failing. We should track down why it was failing, and restore the test. **Done is:** * `$this->markTestSkipped()` is...

danielbachhuber