8000 Inline canRunForked() check in DesignerNewElementWizard by ptziegler · Pull Request #1329 · eclipse-windowbuilder/windowbuilder · GitHub
[go: up one dir, main page]

Skip to content

Inline canRunForked() check in DesignerNewElementWizard#1329

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:cleanup-new-element-wizard
Dec 22, 2025
Merged

Inline canRunForked() check in DesignerNewElementWizard#1329
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:cleanup-new-element-wizard

Conversation

@ptziegler
Copy link
Contributor

This method is defined in an internal class, is never overridden and therefore always returns true. Given that it is only used in one place, we can use this value directly when executing the workspace runnable.

@ptziegler ptziegler added this to the 1.23.0 milestone Dec 21, 2025
@github-actions
Copy link
github-actions bot commented Dec 21, 2025

Test Results

 2 720 files  + 7   2 720 suites  +7   10h 51m 57s ⏱️ + 10m 42s
 7 004 tests ± 0   6 946 ✅ + 1   58 💤 ±0  0 ❌ ±0 
28 016 runs  +36  27 784 ✅ +37  232 💤 ±0  0 ❌ ±0 

Results for commit bd91e6a. ± Comparison against base commit a44ab90.

♻️ This comment has been updated with latest results.

This method is defined in an internal class, is never overridden and
therefore always returns `true`. Given that it is only used in one
place, we can use this value directly when executing the workspace
runnable.
@ptziegler ptziegler force-pushed the cleanup-new-element-wizard branch from 5894482 to bd91e6a Compare December 21, 2025 21:41
@ptziegler ptziegler merged commit adfa806 into eclipse-windowbuilder:master Dec 22, 2025
8 checks passed
@ptziegler ptziegler deleted the cleanup-new-element-wizard branch December 22, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0