childrenIgnored wrong return?

[pilaoda]

node-glob/src/ignore.ts

Lines 109 to 111 in ed22c06

	for (const m of this.absoluteChildren) {
	if (m.match(fullpath)) true
	}

Maybe I misunderstanding the purpose here, shouldn't this block return true rather single true?

[pilaoda]

I run into a problem that the ignore option doesn't ignore a infinite symlink directory and iterate its infinite files. I think the problem is this code which doesn't work as expected.

[isaacs]

Omg, yes, that is obviously a bug, good spot.

What's funny is that it actually still works, assuming the path isn't an ELOOP, just way slower because it iterates over every file.