Is this change fixing the problem where argument values were not passed through?

This implementation would be easier to reason about:

```js
} else {
  result.push('--' + key);
  result.push(value);
}
```