contra

@Yimura A PR would have worked too, no need to create a whole new project. This one is actively maintained by me. The implementation plan to solve this ticket is...

@Bara-dev I'm not the original author of this module - I stepped in to maintain it and be responsive to PRs + answer issues and keep it up to date....

We could add an onError option that would wrap https://github.com/aseemk/requireDir/blob/master/index.js#L128 in a try/catch - why do you need to catch it per-file?

It seems like a fine option to add and should only add a few lines of code - feel free to send a PR that includes docs + tests.

@pezza3434 Could you rebase this? I'll try to get this merged and published ASAP. Thanks for the contribution.

@Janpot I agree, I think this should be the default behavior (recurse or no recurse). As for breaking code, I'll publish it as a breaking release with a note in...

ping @pezza3434

Is this a feature you're still interested in? I'd take a PR for it if it doesn't add too much surface and is done in a generic way.

@jlgrall Great, looking forward to a PR from you (or somebody else reading this) - it shouldn't be too hard to replace the `key = require(path)` with `key = path`...

Is this a feature you're still interested in? I'd take a PR for it if it doesn't add too much surface and is done in a generic way.