Joe Bateson

I went ahead and added basic support for this on https://github.com/jdb8/znc-push/tree/arbitrary-command-notifications - I don't normally work with C++ so it's not great. Wanted to post this here if anyone else...

That makes sense, I didn't consider the public bouncer use-case. I'll create the PR in any case to start with.

It's possible related to #9 (on the server side), which I'd rather look at instead because it's affecting even small folders (so definitely not the inotify limits this time). Thanks...

Which version of node is producing this error? Can you paste the full traceback if there's more? I can't tell what's actually throwing the error here.

Oh, this is running in a browser environment? I wonder if we need to wrap the import-level execution in a check for that?

@santiagoalmeidabolannos just for additional clarification, do you see this error in the browser console at runtime or in a warning at build-time? Just curious for future reference, and also to...

So if I understand correctly, this issue is happening because the code is being imported outside of a node environment: and my change introduced an import-time side effect which assumes...

@jcchavezs @santiagoalmeidabolannos I don't have a ton of time at the moment to work on a fix for this, but it sounds like the problem is well-understood (needing to wrap...

FWIW I published a fork at `@jdb8/eslint-plugin-monorepo` to npm if it's useful: https://github.com/jdb8/eslint-plugin-monorepo It contains this PR and https://github.com/azz/eslint-plugin-monorepo/pull/29, which should provide some additional perf improvements.

@alangpierce thank you for the very detailed explanation! I'll try out what you suggested. I was also able to make things work in the specific case I ran into by...