Remove unused imports#252
Conversation
This commit removes various unused imports across the codebase, as identified by PMD analysis. Care was taken to preserve wildcard imports, which PMD occasionally flags incorrectly but are necessary to avoid compilation failures (particularly for static method imports in test classes). No unused private fields, methods, or variables were found. Co-authored-by: RoiSoleil <3462260+RoiSoleil@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #252 +/- ##
============================================
+ Coverage 73.28% 73.39% +0.11%
- Complexity 3234 3242 +8
============================================
Files 420 421 +1
Lines 14435 14467 +32
Branches 1266 1267 +1
============================================
+ Hits 10578 10618 +40
+ Misses 3325 3316 -9
- Partials 532 533 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I ran a PMD analysis using the
category/java/bestpractices.xml/UnusedImportsruleset (along withUnusedPrivateField,UnusedPrivateMethod, andUnusedLocalVariablewhich turned up nothing) and removed the unused imports flagged across the project.I wrote a python script to strip them out automatically, but excluded lines containing
.*to prevent compilation breakage from false-positive wildcard reports. Tests and builds pass correctly.PR created automatically by Jules for task 569599873569338136 started by @RoiSoleil