Useless deepmerge

[zdm]

Hi.
Are you sure, that deepmerge is needed here:

    async launch(options) {
        // Ensure there are certain properties (e.g. the `options.args` array)
        const defaultLaunchOptions = { args: [] };
        options = merge(defaultLaunchOptions, options);

Why not just write:

async launch(options = {}) {
    if (!options.args) options.args = [];

I am asking, because deepmerge works incorrectly in my case. I need to pass options.proxy as instance of Proxy class, but deepmerge converts it to the plain object.

[zdm]

Or do not merge, if object is not a plain object:

const isPlainObject = require('is-plain-object')
merge(target, source, {
    isMergeableObject: isPlainObject
});

[Niek]

Feel free to submit a PR, but please be aware that if (!options.args) options.args = []; is something very different than merging the two option arrays.

[zdm]

PR
#322

[Niek]

Closing this in favor of #322