-
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
#36: Tool Commandlet for tomcat #250
#36: Tool Commandlet for tomcat #250
Conversation
…re/131-support-for-tool-dependecies
…thub.com/aBega2000/IDEasy into feature/131-support-for-tool-dependecies # Conflicts: # cli/src/main/java/com/devonfw/tools/ide/tool/ToolCommandlet.java
…thub.com/aBega2000/IDEasy into feature/131-support-for-tool-dependecies
…thub.com/aBega2000/IDEasy into feature/131-support-for-tool-dependecies
…131-support-for-tool-dependecies # Conflicts: # cli/src/main/java/com/devonfw/tools/ide/tool/LocalToolCommandlet.java
…thub.com/aBega2000/IDEasy into feature/131-support-for-tool-dependecies
…re/131-support-for-tool-dependecies
# Conflicts: # cli/src/main/java/com/devonfw/tools/ide/tool/LocalToolCommandlet.java
- Includes the tests for tool dependencies
…b.com/aBega2000/IDEasy into feature/36-tool-commandlet-for-tomcat
- Added recursive dependency installation - Added setup of environment variables for dependencies
# Conflicts: # cli/src/main/resources/nls/Ide.properties # cli/src/main/resources/nls/Ide_de.properties
# Conflicts: # cli/src/main/java/com/devonfw/tools/ide/tool/LocalToolCommandlet.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aBega2000 thanks for taking this complex story further. You already made some good progress 👍
However, I still think that this story is too complex to get it solved via review comment feedback ping-pong.
IMHO we need to arrange a screen-sharing session and get this solved together to come to an end in a more efficient way.
# Conflicts: # cli/src/main/resources/nls/Help.properties # cli/src/main/resources/nls/Help_de.properties
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aBega2000 I reviewed and tested your latest code of this PR and tomcat is working. Great work👍
For the lots of review comments, that we discussed in a personal meeting, I created issue #439 for the review rework and therefore resolve the review comments and merge this as is.
incomplete, may not compile, just the state from the review meeting
fixes #36, fixes #131
This is a pull request where Tomcat is implemented and tested.
Additionally the support for tool dependencies is implemented and tested.