genericworkflownodes / GenericKnimeNodes

Base package for GenericKnimeNodes
https://github.com/genericworkflownodes/GenericKnimeNodes
Other
15 stars 16 forks source link

[FEATURE] Adds category group #213

Open rrahn opened 5 years ago

rrahn commented 5 years ago

Allows external configuration systems to boostrap plugins from different providers into a core plugin for distribution.

rrahn commented 5 years ago

@jpfeuffer @AlexanderFillbrunn can you point me to the tests, so I can add one for the feature. And to give you a scenario. I want to be able to ship in one plugin version sources from multiple vendors. To show that some of the tools are SeqAn and others from external vendors I can add this on a configuration level and highlight it visually.

AlexanderFillbrunn commented 5 years ago

Makes sense. But maybe the default should be for getCategoryGroup to return null, so people do not have to implement it explicitly and only overwrite it if necessary. I am afraid we don't have any tests for this yet, unless @jpfeuffer wrote some.

rrahn commented 5 years ago

It makes sense to make it not abstract. I'll change this.

jpfeuffer commented 5 years ago

There are some rudimentary tests in the com.gkn. ... . test package.