Skip to content
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

Mark unused methods for removal in WorkbenchWindowAdvisor #2501

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vogella
Copy link
Contributor

@vogella vogella commented Nov 6, 2024

These methods are not used since the 3.x migration, time to mark them for deletion.

Copy link
Contributor

github-actions bot commented Nov 6, 2024

Test Results

 1 821 files  ±0   1 821 suites  ±0   1h 51m 59s ⏱️ - 3m 30s
 7 725 tests ±0   7 496 ✅ ±0  228 💤 ±0  1 ❌ ±0 
24 336 runs  ±0  23 588 ✅ ±0  747 💤 ±0  1 ❌ ±0 

For more details on these failures, see this check.

Results for commit 560c490. ± Comparison against base commit beb9918.

♻️ This comment has been updated with latest results.

@vogella vogella force-pushed the workbench-window-advisor-mark-methods-for-deletion branch from f5f77ed to 00f34d6 Compare November 6, 2024 16:12
@vogella
Copy link
Contributor Author

vogella commented Nov 6, 2024

Awesome this new quality gate check! This prevented me from adding more warnings to the build. I assume I have to thank you @laeubi for this. Great work!

@vogella vogella force-pushed the workbench-window-advisor-mark-methods-for-deletion branch from 00f34d6 to d235146 Compare November 8, 2024 09:58
These methods are not used since the 3.x migration, time to mark them
for deletion.
@vogella vogella force-pushed the workbench-window-advisor-mark-methods-for-deletion branch from d235146 to 560c490 Compare November 11, 2024 08:53
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