Skip to content

#2400 Rework console - #2487

Draft
laim2003 wants to merge 28 commits into
devonfw:mainfrom
laim2003:feature/#2400-rework-console
Draft

laim2003 wants to merge 28 commits into
devonfw:mainfrom
laim2003:feature/#2400-rework-console

Conversation

@laim2003

@laim2003 laim2003 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

This PR fixes #2400

BLOCKED BY: #2433 / PR #2481

Implemented changes:

  • Migrated console from MVC to MVVM pattern
  • Introduced log handling via the new EventBus

Testing instructions

Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:


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 and not feature/921 fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summaries 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 labelled
    with internal
  • You have not changed any dependency in pom.xml files or otherwise if runtime dependencies changed, you have updated our LICENSE.asciidoc
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

@laim2003 laim2003 moved this from 🆕 New to 🏗 In progress in IDEasy board Sep 16, 2026
…leViewModel

# Conflicts:
#	gui/src/main/java/com/devonfw/ide/gui/App.java
#	gui/src/main/java/com/devonfw/ide/gui/factory/TabFactory.java
#	gui/src/main/java/com/devonfw/ide/gui/ui/mainwindow/MainWindowView.java
#	gui/src/test/java/com/devonfw/ide/gui/AppBaseTest.java
#	gui/src/test/java/com/devonfw/ide/gui/factory/TabFactoryTest.java
- reformatted ConsoleViewModel
@laim2003
laim2003 force-pushed the feature/#2400-rework-console branch from e1bae58 to d960270 Compare September 16, 2026 15:55
@laim2003 laim2003 added GUI Graphical User Interface of IDEasy (aka dashboard) build with JavaFx internal Nothing to be added to CHANGELOG, only internal story labels Sep 16, 2026
…ole' into feature/devonfw#2400-rework-console

# Conflicts:
#	gui/src/main/java/com/devonfw/ide/gui/ui/mainwindow/MainWindowView.java
Signed-off-by: laim2003 <luk.faber@gmx.de>
Signed-off-by: laim2003 <luk.faber@gmx.de>
…ole' into feature/devonfw#2400-rework-console

# Conflicts:
#	gui/src/main/java/com/devonfw/ide/gui/ui/mainwindow/MainWindowView.java
@coveralls

coveralls commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 35579185472

Warning

No base build found for commit e224e69 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 74.385%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 19264
Covered Lines: 14971
Line Coverage: 77.71%
Relevant Branches: 8560
Covered Branches: 5726
Branch Coverage: 66.89%
Branches in Coverage %: Yes
Coverage Strength: 3.33 hits per line

💛 - Coveralls

@laim2003

Copy link
Copy Markdown
Contributor Author

I'll wait until #2401 is merged so the diff in this PR updates and becomes less chaotic

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

Labels

GUI Graphical User Interface of IDEasy (aka dashboard) build with JavaFx internal Nothing to be added to CHANGELOG, only internal story

Projects

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

Rework Console to MVVM

3 participants