Skip to content

#1968: Launch GUI in background#1984

Merged
hohwille merged 6 commits into
devonfw:mainfrom
laim2003:fix/1968-start-gui-in-background
May 29, 2026
Merged

#1968: Launch GUI in background#1984
hohwille merged 6 commits into
devonfw:mainfrom
laim2003:fix/1968-start-gui-in-background

Conversation

@laim2003
Copy link
Copy Markdown
Contributor

@laim2003 laim2003 commented May 29, 2026

This PR fixes #1968

Implemented changes:

  • Changed PROCESS_MODE for the maven tool from DEFAULT to BACKGROUND_SILENT. This launches the GUI without blocking the terminal.

Testing instructions

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

  1. Create a GUI Commandlet Execution in your IDE
  2. Execute it
  3. GUI should only show a few log messages, then the output should terminate and the GUI should pop up.

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
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board May 29, 2026
@laim2003 laim2003 self-assigned this May 29, 2026
@laim2003 laim2003 added GUI Graphical User Interface of IDEasy (aka dashboard) build with JavaFx process executing external programs (ProcessContext) internal Nothing to be added to CHANGELOG, only internal story labels May 29, 2026
@laim2003 laim2003 moved this from 🆕 New to 🏗 In progress in IDEasy board May 29, 2026
@laim2003 laim2003 added this to the release:2026.06.001 milestone May 29, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented May 29, 2026

Coverage Report for CI Build 26650357092

Coverage increased (+0.009%) to 71.097%

Details

  • Coverage increased (+0.009%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 15710
Covered Lines: 11650
Line Coverage: 74.16%
Relevant Branches: 7004
Covered Branches: 4499
Branch Coverage: 64.23%
Branches in Coverage %: Yes
Coverage Strength: 3.14 hits per line

💛 - Coveralls

@laim2003 laim2003 moved this from 🏗 In progress to Team Review in IDEasy board May 29, 2026
@laim2003 laim2003 marked this pull request as ready for review May 29, 2026 09:39
@AdemZarrouki AdemZarrouki self-assigned this May 29, 2026
@AdemZarrouki AdemZarrouki self-requested a review May 29, 2026 10:23
Comment thread cli/src/main/package/gui/pom.xml
@laim2003 laim2003 moved this from Team Review to 👀 In review in IDEasy board May 29, 2026
@laim2003 laim2003 requested a review from hohwille May 29, 2026 11:35
@laim2003 laim2003 marked this pull request as draft May 29, 2026 15:54
@laim2003
Copy link
Copy Markdown
Contributor Author

Made a little mistake while resolving a merge conflict... will clean this up quickly and then mark it as ready to merge again

@laim2003 laim2003 removed the internal Nothing to be added to CHANGELOG, only internal story label May 29, 2026
@laim2003 laim2003 marked this pull request as ready for review May 29, 2026 16:00
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.

@laim2003 thanks for this PR. Besides running in background you also fixed a bug that you already fixed in PR #1967 and I did not understand it during the review and since time was running out before the release I just reverted your change causing the bug again.
Sorry for that and great that you fixed it again. 👍
Ready for merge.

@hohwille hohwille merged commit d815e54 into devonfw:main May 29, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in IDEasy board May 29, 2026
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 process executing external programs (ProcessContext)

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Gui should start in background

4 participants