Exclude copying 'core/node_modules' in getCodebaseFinder

Won't pretend I know know that system but the idea of excluding node_modules makes sense.

Yes this just looks like an oversight.

I started wondering whether we should try to use (or find a way to add a common default for) $settings['file_scan_ignore_directories'] but then persuaded myself that just hardcoding this here is fine.

Spokje → closed merge request !5122

Automatically closed - issue fixed for 2 weeks with no activity.