Improve install size

[szmarczak]

https://packagephobia.com/result?p=tempy

[email protected] install size is 1.61 MB

That's quite high. 0.6.0 was only 120KB.

[sindresorhus]

I agree, however, I have a plan. When the recursive option on rmdir is no longer experimental, we can use that one instead of del: https://nodejs.org/api/fs.html#fs_fs_rmdir_path_options_callback

---

Tip: You can use https://bundlephobia.com/[email protected] to see what dependencies take up of the most space.

[sindresorhus]

I managed to improve it a little bit: https://packagephobia.com/result?p=tempy

[Richienb]

When the recursive option on rmdir is no longer experimental

That's Node.js 12.10. Node.js 10 will EOL in 1 month and 2 weeks.

[fregante]

Just in time for sindresorhus/meta#15!

Side note: Could fs.mkdtemp be used to replace unique-string?

I think from

tempy/index.js

Line 14 in 0e498c9

const getPath = (prefix = '') => path.join(tempDir, prefix + uniqueString());

to something like

const getPath = (prefix = '') => fs.mkdtempSync(path.join(tempDir, prefix));

[Richienb]

I agree, however, I have a plan. When the recursive option on rmdir is no longer experimental, we can use that one instead of del: nodejs.org/api/fs.html#fs_fs_rmdir_path_options_callback

That option is deprecated now.

The alternative is fs.promises.rm in Node.js 14.14 which is 3 months away.

[Richienb]

Side note: Could fs.mkdtemp be used to replace unique-string?

Yes it would, though we'd still need to use unique-string to generate the file since that isn't a feature yet