-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix Array to string conversion in TsvFormatter.php on line 35 #3721 #3917
Conversation
At minimum this will need to update composer.lock, Waiting for @greg-1-anderson for a code review. |
I updated composer.lock, but the tests are failing. See #3918. To update composer.lock here, just run |
@@ -17,7 +17,7 @@ class BatchCommands extends DrushCommands | |||
* | |||
* @return \Consolidation\OutputFormatters\StructuredData\UnstructuredListData | |||
*/ | |||
public function process($batch_id, $options = ['format' => 'json']) | |||
public function process($batch_id, $options = ['format' => 'null']) |
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 caller expects JSON to be returned. See
Lines 152 to 153 in 92b2359
$process->run($process->showRealtime()->hideStdout()); | |
$result = $process->getOutputAsJson(); |
@@ -37,7 +37,7 @@ | |||
"consolidation/annotated-command": "^2.10.2", | |||
"consolidation/config": "^1.1.0", | |||
"consolidation/filter-via-dot-access-data": "^1", | |||
"consolidation/output-formatters": "^3.3.1", | |||
"consolidation/output-formatters": "^3.4", |
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.
This does look useful. After you run composer update, please commit the changes that happen in .scenario directory.
Drush now depends on output-formatters ^3.4. |
See 3721