unicorn/no-array-push-push - breaks stream.push

unicorn/no-array-push-push combines .push, but it can’t do this if the .push is for something that is not an array, like stream.

Before

const stream = new Readable();
stream.push('attachment content');
stream.push(null); // end of file

After

const stream = new Readable();
stream.push('attachment content', null); // end of file
//                                ^^^^ null is not a valid encoding

[sindresorhus]

But you have to end the stream with null. Alternatively, we could just ignore if the instance is called stream.

[fisker]

I understand that, my point is doing this is not really very useful,

const stream = new Readable();
stream.push('one string');
stream.push('another string');  // <- this will be reported too