Skip to content

#1688: Remove unnecessary message in the CLI when installing a new tool#1863

Open
jakozian wants to merge 4 commits intodevonfw:mainfrom
jakozian:fix/#1688-remove-unnecessary-cli-message-install-tool
Open

#1688: Remove unnecessary message in the CLI when installing a new tool#1863
jakozian wants to merge 4 commits intodevonfw:mainfrom
jakozian:fix/#1688-remove-unnecessary-cli-message-install-tool

Conversation

@jakozian
Copy link
Copy Markdown
Contributor

@jakozian jakozian commented Apr 28, 2026

This PR fixes #1688

Implemented changes:

  • Remove unnecessary log messages

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summarizes what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
    with internal

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board Apr 28, 2026
@jakozian jakozian added logging IDEasy specific logging (levels, messages, etc.) install installation process of IDE + tools and install commandlet labels Apr 28, 2026
@jakozian jakozian self-assigned this Apr 28, 2026
@jakozian jakozian moved this from 🆕 New to Team Review in IDEasy board Apr 28, 2026
@jakozian jakozian moved this from Team Review to 👀 In review in IDEasy board Apr 28, 2026
@jakozian jakozian requested a review from hohwille April 28, 2026 07:17
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 28, 2026

Coverage Report for CI Build 25376709217

Coverage increased (+0.04%) to 70.764%

Details

  • Coverage increased (+0.04%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 25 coverage regressions across 3 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

25 previously-covered lines in 3 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/tool/docker/Docker.java 18 6.02%
com/devonfw/tools/ide/tool/GlobalToolCommandlet.java 6 2.34%
com/devonfw/tools/ide/version/VersionSegment.java 1 90.55%

Coverage Stats

Coverage Status
Relevant Lines: 15399
Covered Lines: 11356
Line Coverage: 73.75%
Relevant Branches: 6878
Covered Branches: 4408
Branch Coverage: 64.09%
Branches in Coverage %: Yes
Coverage Strength: 3.12 hits per line

💛 - Coveralls

@hohwille hohwille added this to the release:2026.05.001 milestone May 4, 2026
Copy link
Copy Markdown
Member

@hohwille hohwille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jakozian thanks for your PR and this quickfix for the main issue. 👍
Ready for merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

install installation process of IDE + tools and install commandlet logging IDEasy specific logging (levels, messages, etc.)

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

Remove unnecessary message in the CLI when installing a new tool

3 participants