-
Notifications
You must be signed in to change notification settings - Fork 351
[WIP] Add dependency check for modules/plugins #558
base: main
Are you sure you want to change the base?
Conversation
43064f9
to
8f24f95
Compare
Codecov Report
@@ Coverage Diff @@
## master #558 +/- ##
==========================================
- Coverage 38.64% 38.28% -0.36%
==========================================
Files 82 82
Lines 10081 10157 +76
==========================================
- Hits 3896 3889 -7
- Misses 6185 6268 +83
Continue to review full report at Codecov.
|
|
2427676
to
7df6146
Compare
Should I merge it, or is it still WIP? |
I like it. |
nice to hear that.. :-)
I think the "backend" code is pretty much complete.. but I'm not entirely happy with the way the information is presented to the user.. but I haven't come up with a nicer idea. Additionally I think we should identify all the dependency for the modules before merging into master... at the moment this is more a PoC than a full (documented) PR... I added the tasks as I see them and will try to find some time the next days to work on this (thanks for the reminder.. ;-) ) |
I still like this feature, but conflicts need to be resolved. |
I am still keen on reviewing whether this would be useful. Particularly now that I split the modules away from the core and into a dedicated repository. |
This PR adds the possibility to add the following dependencies to Modules:
[]
) - a list of python packages that this Module needs[]
) - a list of system binaries that this Module needsToDos