Some functions from Lodash can be replaced with native ones

[live627]

For example, lodash/flatten can be replaced with Array.prototype.flat() and even combined with Array.prototype.map() by using Array.prototype.flatMap().

Example:

ava/lib/line-numbers.js

Lines 52 to 58 in e8a4976

	export function getApplicableLineNumbers(normalizedFilePath, filter) {
	return sortNumbersAscending(distinctArray(flatten(
	filter
	.filter(({pattern, lineNumbers}) => lineNumbers && picomatch.isMatch(normalizedFilePath, pattern))
	.map(({lineNumbers}) => lineNumbers),
	)));
	}

 export function getApplicableLineNumbers(normalizedFilePath, filter) {
-	return sortNumbersAscending(distinctArray(flatten(
+	return sortNumbersAscending(distinctArray(
 		filter
 			.filter(({pattern, lineNumbers}) => lineNumbers && picomatch.isMatch(normalizedFilePath, pattern))
-			.map(({lineNumbers}) => lineNumbers),
-	)));
+			.flatMap(({lineNumbers}) => lineNumbers),
+	));
 }

Would this be something that is of interest? Eventually being able to remove Lodash would save ~1.3 MB (over 2.2 MB on my disk)

Here's a good resource

[novemberborn]

Yes that'd be great.

[live627]

That's good to hear . I wasn't sure of the reaction especially after seeing this comment and how it was closed for being "too controversial", whatever that means. Maybe it was the aggressive attitude. Anyway, I agree with the sentiment that this packages should have a smaller footprint and less dependencies if at all possible.