Skip to content

Clean Code for apitools/org.eclipse.pde.api.tools.ui#1669

Merged
laeubi merged 1 commit intomasterfrom
clean-code/apitools/org.eclipse.pde.api.tools.ui
Mar 18, 2025
Merged

Clean Code for apitools/org.eclipse.pde.api.tools.ui#1669
laeubi merged 1 commit intomasterfrom
clean-code/apitools/org.eclipse.pde.api.tools.ui

Conversation

@eclipse-pde-bot
Copy link
Contributor

The following cleanups where applied:

  • Add final modifier to private fields
  • Add missing '@Deprecated' annotations
  • Add missing '@Override' annotations
  • Add missing '@Override' annotations to implementations of interface methods
  • Convert control statement bodies to block
  • Make inner classes static where possible
  • Replace deprecated calls with inlined content where possible
  • Use pattern matching for instanceof

@github-actions
Copy link

github-actions bot commented Mar 17, 2025

Test Results

   285 files  +  6     285 suites  +6   48m 19s ⏱️ + 2m 41s
 3 608 tests ±  0   3 532 ✅ +  1   76 💤 ± 0  0 ❌  - 1 
11 016 runs  +194  10 785 ✅ +166  231 💤 +29  0 ❌  - 1 

Results for commit 69be128. ± Comparison against base commit 874e43f.

♻️ This comment has been updated with latest results.

@eclipse-pde-bot eclipse-pde-bot force-pushed the clean-code/apitools/org.eclipse.pde.api.tools.ui branch from 713b73c to 69be128 Compare March 18, 2025 02:33
@laeubi laeubi merged commit 4ec80e5 into master Mar 18, 2025
18 checks passed
@akurtakov akurtakov deleted the clean-code/apitools/org.eclipse.pde.api.tools.ui branch March 18, 2025 06:42
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.

2 participants