- 
        Couldn't load subscription status. 
- Fork 45
Description
Story #439 was again rather complex since all was done in a single PR #515.
I finally wanted to get this merged. However, one concern still needs to be resolved from #515:
IDEasy/cli/src/main/java/com/devonfw/tools/ide/tool/ToolCommandlet.java
Lines 98 to 102 in 86df4af
| * @param existsEnvironmentContext the {@link Boolean} that checks if an environment context exits. If true, then the process context is passed to the | |
| * installation method as an argument. | |
| * @param args the command-line arguments to run the tool. | |
| */ | |
| public void runTool(ProcessMode processMode, VersionIdentifier toolVersion, boolean existsEnvironmentContext, String... args) { | 
This parameter existsEnvironmentContext is very strange and other developers will not understand if they need to pass true or false here. IMHO this parameter must be removed and it seems more related to the fact if a tool has dependencies or not what can and should be determined automatically by the presence or absence of a dependencies.json file.
Also I think this is related to #556 and maybe we can simplify this after fixing bug #556 and then always create ProcessContext and pass it upfront.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status