Disallow static import of com.google.errorprone.{VisitorState,util.ASTHelpers} members

[rickie]

Saw this during a code review 😄.

Probably we can add more from Error Prone later. I checked some other classes from Error Prone but these are clear improvements that we also have been discussing before. Therefore, I decided to start with these two.

[github-actions]

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

[Stephan202]

Rebased and added a commit. At least one Error Prone check has a suggested fix in which an ASTHelpers method is statically imported, so it's nice that this is now auto-fixed.

Suggested commit message:

Disallow static import of `com.google.errorprone.{VisitorState,util.ASTHelpers}` members (#922)

[Stephan202]

To match the GJF's import order:

Suggested change

	"com.google.errorprone.util.ASTHelpers",
	"com.google.errorprone.VisitorState",
	"com.google.errorprone.VisitorState",
	"com.google.errorprone.util.ASTHelpers",

[github-actions]

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

[github-actions]

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud