Skip to content

Clean Code for apitools/org.eclipse.pde.api.tools.tests#1750

Merged
akurtakov merged 1 commit into
masterfrom
clean-code/apitools/org.eclipse.pde.api.tools.tests
May 6, 2025
Merged

Clean Code for apitools/org.eclipse.pde.api.tools.tests#1750
akurtakov merged 1 commit into
masterfrom
clean-code/apitools/org.eclipse.pde.api.tools.tests

Conversation

@eclipse-pde-bot

Copy link
Copy Markdown
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
  • Remove unnecessary array creation for varargs
  • Replace deprecated calls with inlined content where possible
  • Use pattern matching for instanceof

@github-actions

github-actions Bot commented May 6, 2025

Copy link
Copy Markdown

Test Results

   285 files  ±0     285 suites  ±0   47m 31s ⏱️ - 2m 46s
 3 611 tests ±0   3 534 ✅  - 1   76 💤 ±0  1 ❌ +1 
11 025 runs  ±0  10 793 ✅  - 1  231 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 6dacaaf. ± Comparison against base commit 4873af8.

@akurtakov akurtakov merged commit df97842 into master May 6, 2025
17 of 19 checks passed
@akurtakov akurtakov deleted the clean-code/apitools/org.eclipse.pde.api.tools.tests branch May 6, 2025 04:34
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