checkstyle / checkstyle/checkstyle

Minimize diff in google style inputs

Open
#17,448 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
9.6k
Forks
4.2k
Avg merge
22h 23m
Merged PRs (30d)
232

Description

Follow up after fix for initial implementation https://github.com/checkstyle/checkstyle/issues/17258

Suppression list were added at
https://github.com/checkstyle/checkstyle/blob/b24679c59d363d263156859fde87e488a5cb8b39/.ci/google-java-format.sh#L12-L15

We need to investigate if that is possible to avoid or minimize suppressions.
Original idea was to keep majority under 50 bytes diff, we should investigate ability to use it.

Split of files into few might be also a strategy.

Contributor guide

Open the contributing guide

Research direction

Start with .ci/google-java-format.sh lines 12-15 and the follow-up context in issue 17258. Investigate whether the current suppression list can be reduced while keeping most changes under a 50-byte diff, and consider splitting files as mentioned. Done means documenting or implementing a reduced-suppression approach with an acceptable formatting diff.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, shell
Domain
ci-cd, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.