Skip to content

Clean Code for bundles/org.eclipse.text #3406

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 17, 2025
Merged

Clean Code for bundles/org.eclipse.text #3406

merged 1 commit into from
Oct 17, 2025

Conversation

@eclipse-platform-bot
Copy link
Contributor

@eclipse-platform-bot eclipse-platform-bot commented Oct 17, 2025

The following cleanups were 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 trailing white spaces on all lines
  • Remove unnecessary array creation for varargs
  • Remove unnecessary suppress warning tokens
  • Remove unused imports
  • Remove unused private constructors
  • Remove unused private fields
  • Remove unused private methods
  • Remove unused private types
  • Replace deprecated calls with inlined content where possible
  • Use pattern matching for instanceof

@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2025

Test Results

 3 018 files  ±0   3 018 suites  ±0   2h 26m 10s ⏱️ +59s
 8 226 tests ±0   7 977 ✅ +1  249 💤 ±0  0 ❌  - 1 
23 598 runs  ±0  22 804 ✅ +1  794 💤 ±0  0 ❌  - 1 

Results for commit cfc64c6. ± Comparison against base commit 4f771f0.

@akurtakov akurtakov merged commit 5b3382d into master Oct 17, 2025
18 checks passed
@akurtakov akurtakov deleted the clean-code/bundles/org.eclipse.text branch October 17, 2025 07:35
akurtakov added a commit to akurtakov/eclipse.platform.ui that referenced this pull request Oct 17, 2025
@akurtakov
Enable save actions for o.e.text
2f1beb9 
PR eclipse-platform#3399
introduced trailing whitespaces and PR
eclipse-platform#3406 was
created by the bot to clean this up.
With save actions we should prevent such cases.
@akurtakov akurtakov mentioned this pull request Oct 17, 2025
Merged
akurtakov added a commit that referenced this pull request Oct 17, 2025
@akurtakov
Enable save actions for o.e.text
301653e 
PR #3399
introduced trailing whitespaces and PR
#3406 was
created by the bot to clean this up.
With save actions we should prevent such cases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cleanup

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eclipse-platform-bot @akurtakov