Remove redundant Checkstyle rules covered by ErrorProne
Open
build
improvement
P3
- Dominant language
- Java
- Stars
- 8.7k
- Forks
- 4.7k
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 196
Description
Now that ErrorProne is enabled across all projects, we can remove redundant checks from other static analysis. I noticed there are at least some redundant checks in Checkstyle, for example `MissingDeprecated`.
Imported from Jira [BEAM-4668](https://issues.apache.org/jira/browse/BEAM-4668). Original Jira may contain additional context.
Reported by: swegner.
Contributor guide
Research direction
Start by locating the Checkstyle configuration across the projects and review the MissingDeprecated rule as the initial example. Compare the configured checks with ErrorProne, then verify that redundant Checkstyle rules are removed while the remaining static analysis coverage is preserved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100