use rewrite fallback, drop oboslete use rewrite fallback, drop oboslete checkstyle``

[Pankraz76]

---

I hereby agree to the terms of the JUnit Contributor License Agreement.

---

Definition of Done

• There are no TODOs left in the code
• Method preconditions are checked and documented in the method's Javadoc
• Coding conventions (e.g. for logging) have been followed
• Change is covered by automated tests including corner cases, errors, and exception handling
• Public API has Javadoc and @API annotations
• Change is documented in the User Guide and Release Notes

[Pankraz76]

issue:

Starting a Gradle Daemon, 1 stopped Daemon could not be reused, use --status for details
Daemon JVM discovery is an incubating feature.
Type-safe project accessors is an incubating feature.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'junit-framework'.
> Task with name 'clean' not found in root project 'junit-framework'.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Get more help at https://help.gradle.org.

BUILD FAILED in 29s
32 actionable tasks: 5 executed, 27 up-to-date
➜  junit-framework git:(drop-checkstyle-pr) 

[Pankraz76]

issue:

FAILURE: Build failed with an exception.

• What went wrong:
  A problem occurred configuring root project 'junit-framework'.

Task with name 'clean' not found in root project 'junit-framework'.

@marcphilipp

could you please give me a hint on how to remove this plugin as well? Spotless removal was possible, so why this one not?

#3

Thanks.

[marcphilipp]

@Pankraz76 It's not clear to me what you're trying to accomplish. Please stop working on things nobody from the JUnit team has asked for. I will no longer respond to notifications from your repository because I'm convinced my energy is better spent elsewhere.

[Pankraz76]

Im not asking to merge, its just an learning experience and supposed to be an demonstrator only.

My impression was that its not possible due to the technical limitations and they might seem not to be hard.

But talking about this and doing the integration on PMD there is actually no need to remove check, as its stilly nice to serve as a second layer.

It should be easy to integrate, even tho thats not quite the case when applying the config. Maybe its working on junit, so the only replacement would be spot to rewrite, as the equals avoid is not done by spot ATM.

• https://docs.openrewrite.org/running-recipes/popular-recipe-guides/automatically-fix-checkstyle-violations
• Fix code cleanup pmd#3

[Pankraz76]

trying to accomplish

I want to learn and explore. It seems the changes made regarding equalsAvoidNull are inconsistent.