Fix error caused by the absence of files when starting gulp task

[KolinRobinson]

If there are no files in the folder, then task
.pipe (size ({
'gzip': true,
'pretty': true,
'showFiles': true,
'showTotal': true
}))
gives an error message

TypeError: Expected a finite number, got undefined: undefined
at module.exports (W: \ Freelance \ Easy-webdev-startpack \ node_modules \ pretty-bytes \ index.js: 70: 9)
at Transform._flush (W: \ Freelance \ Easy-webdev-startpack \ node_modules \ gulp-size \ index.js: 115: 21).

This fix solves this error, allowing you not to write additional functionality (in the form of disabling tasks if the folder with files is empty).

Thanks for attention!

Link to the task on which the error is caused: budfy/Easy-webdev-startpack@50ad1fe#diff-6e5c91cb2762eb968b4beffd0b5f42b94a46c4b6b5400195cb64ed98787a09a2

[sindresorhus]

CI is failing. Always run npm test locally before pushing changes.

[sindresorhus]

The PR also need a proper descriptive title.

[KolinRobinson]

Thanks for the comments. Corrected.

[sindresorhus]

prettySize is documented to always exist, so it should fall back to 0 for this.size. Just do a ternary.

[sindresorhus]

This is not what I asked. You need to add the fallback to this.size, as it's also expected to not be undefined.

[KolinRobinson]

Sorry, this is my first PL.

If there are no files, this.size is undefined, which cannot be processed by the prettyBytes () method, which causes a gulp error. In this case, I see a solution either
this.size = totalSize.values ​​(). next (). value || 0;
or
this.prettySize = prettyBytes (this.size || 0);

[sindresorhus]

this.size = totalSize.values ​​(). next (). value || 0;

This is what I meant.

[KolinRobinson]

Okay. Thanks a lot!