-
Notifications
You must be signed in to change notification settings - Fork 18
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
Create support for IDEs #10
Labels
enhancement
New feature or request
Milestone
Comments
hohwille
added a commit
that referenced
this issue
Sep 20, 2023
This was referenced Oct 13, 2023
hohwille
added a commit
to hohwille/IDEasy
that referenced
this issue
Nov 8, 2023
…gurator refactoring, bugfix with newContext and copy
jan-vcapgemini
added a commit
to hohwille/IDEasy
that referenced
this issue
Nov 9, 2023
jan-vcapgemini
added a commit
to hohwille/IDEasy
that referenced
this issue
Nov 9, 2023
jan-vcapgemini
added a commit
to hohwille/IDEasy
that referenced
this issue
Nov 9, 2023
string concatenation replaced with inline variables
hohwille
added a commit
that referenced
this issue
Nov 16, 2023
Was auto-closed by PR but project import and some other features not completed. Hence I do reopen. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently we have implemented
ToolCommandlet
with subclasses likeJava
orMvn
.In order to implement
Eclipse
,Intellij
, etc. we need an additional abstract class derived fromToolCommandlet
that handles all the features required for IDEs:Configurator
- this is the old code https://github.com/devonfw/ide/blob/af67497e06fb82e2e50a4d9ffa10e3f9dda44aa1/scripts/src/main/resources/scripts/functions#L1538)For the
Configurator
we should:IDEasy/cli/src/main/java/com/devonfw/tools/ide/configurator/Configurator.java
Lines 247 to 252 in ebb06e5
run
method withString[]
args, we should add explicit parameters to the constructor and change therun
method to bepublic void run()
instead. SeeIDEasy/cli/src/main/java/com/devonfw/tools/ide/configurator/Configurator.java
Line 153 in ebb06e5
IdeContext
orIdeLogger
toConfigurator
as Constructor arg and use that instead of the old logger:IDEasy/cli/src/main/java/com/devonfw/tools/ide/configurator/Configurator.java
Line 281 in ebb06e5
The text was updated successfully, but these errors were encountered: