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

Check plugin dependencies on generation #31

Closed
wants to merge 1 commit into from
Closed

Check plugin dependencies on generation #31

wants to merge 1 commit into from

Conversation

sergej-koscejev
Copy link
Contributor

The check_IdeaPlugin_Dependencies rule doesn't take into account
runtimes of used languages because it doesn't have that information in
the model. So if a model uses a language that is not otherwise included
as a dependency, no errors were reported in the editor but this missing
dependency would lead to ClassNotFoundExceptions later on.

The information is available during the generation time after modules
are loaded, so perform a check at that time as a last resort.

The check_IdeaPlugin_Dependencies rule doesn't take into account
runtimes of used languages because it doesn't have that information in
the model.  So if a model uses a language that is not otherwise included
as a dependency, no errors were reported in the editor but this missing
dependency would lead to ClassNotFoundExceptions later on.

The information is available during the generation time after modules
are loaded, so perform a check at that time as a last resort.
@artem-tikhomirov
Copy link
Collaborator

@sergej-koscejev
Copy link
Contributor Author

Will redo this

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