Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add some guards / error messages around missing plugins / wrong profile setup #152

Open
alterNERDtive opened this issue Jul 25, 2022 · 0 comments

Comments

@alterNERDtive
Copy link
Owner

Common problems seem to be not installing EDDI and running the profiles directly, without including anything including the base profile.

Currently commands just assume everything is set up correctly; e.g. there are no checks if EDDI variables exist, they are just used. There are no checks if the base profile is active, commands are just (attempted to) run.

EDDI version exists once that is loaded, the base plugin would need to set its own flag. Checking in every command sounds like a PITA; the individual profiles are still setup to run their own startup command from way back when though. I should probably change that to a new command that just complains about:

  • running profile directly
  • not including alterNERDtive.base
  • potentially not having EDDI

And I guess the base profile’s startup should/could check for all required plugins.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant