Bulk formatting and removing unused imports

[shawkins]

Description

Fix #3984 and enables removing unused imports.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change
• Chore (non-breaking change which doesn't affect codebase;
  test, version modification, documentation, etc.)

Checklist

• Code contributed by me aligns with current project license: Apache 2.0
• I Added CHANGELOG entry regarding this change
• I have implemented unit tests to cover my changes
• I have added/updated the javadocs and other documentation accordingly
• No new bugs, code smells, etc. in SonarCloud report
• I tested my code in Kubernetes
• I tested my code in OpenShift

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
6 Code Smells

39.8% Coverage
6.7% Duplication

[andreaTP]

Style check is failing, apart from that, now that we are at the 6.2 phase I think this does make a lot of sense 🙂
LGTM 👍

[shawkins]

Style check is failing

A few old files have their header comments in the wrong spot, I'll have that addressed shortly.

[andreaTP]

We should remove this line as well:

kubernetes-client/pom.xml

Line 1059 in 9394b5a

<ratchetFrom>origin/master</ratchetFrom>

[shawkins]

We should remove this line as well:

@andreaTP this pr was generated with that removed

[shawkins]

@andreaTP or do you mean that we shouldn't need to have that property anymore and just let it format everything on each run?

[andreaTP]

Yup, correct @shawkins

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
6 Code Smells

35.8% Coverage
7.1% Duplication

[shawkins]

@andreaTP updated the pom to always format - it doesn't make much difference in the build time on my ssd system.

[manusa]