Fix TS definitions

[iamakulov]

The source code uses module.exports = ..., not exports.default = ...:

clean-deep/src/index.js

Lines 14 to 22 in 9d8313d

	module.exports = function cleanDeep(object, {
	cleanKeys = [],
	cleanValues = [],
	emptyArrays = true,
	emptyObjects = true,
	emptyStrings = true,
	nullValues = true,
	undefinedValues = true
	} = {}) {

Given this, TS definitions should use export =, not export default. See microsoft/TypeScript#7185 (comment).

This matters when the project is compiled without the "esModuleInterop": true flag. In that case, doing

import cleanDeep from 'clean-deep'

cleanDeep(...)

would throw an error like TypeError: clean_deep_1.default is not a function.

[FFKL]

@iamakulov cool! What about CleanOptions? Should be moved intonamespace?

[iamakulov]

What about CleanOptions? Should be moved intonamespace?

Dunno. Why?

[FFKL]

Because you overwright exports object by the cleanDeep function and can't use export = with other exports. example

[nunofgs]

Good point @FFKL . Not sure this can be achieved with the = syntax since you can't really export two things as the "default"
(ultimately they need to be attached somewhere).

Closing for now but please do reopen if you have other suggestions. Thanks!