Moerill / fvtt-find-the-culprit

MIT License
8 stars 10 forks source link

Feature Request: enabling dependencies with tested culprit #14

Open franckflo opened 3 years ago

franckflo commented 3 years ago

Thanks so much for this great module, it's saved me a lot of time already 👍. Something helpful would be if there was an option for Find the Culprit to automatically enable the dependencies of the modules when testing through iterations, and when picking the initial modules that should stay enabled. Without those dependencies, the tested modules may not be fully enabled and therefore we wouldn't get the real results.

I also noticed that when choosing the modules to keep enabled, the order is not quite alphabetical (it's like it's going through several lists, each of them in alphabetical order, but not a global alphabetical list. This makes finding the right module difficult when it's in a long list).

diannetea commented 3 years ago

Enabling the dependencies when running is really necessary, I keep running into issues when trying to find a conflict as some modules will not load (or worse cause more issues) if run without their dependencies.