apache / apache/druid

Specify lineWrappingIndentation in Checkstyle's Indentation check

Open
#8,239 3 comments 0 reactions 0 assignees View on GitHub
Area - Automation/Static Analysis Contributions Welcome
Dominant language
Java
Stars
14.1k
Forks
3.8k
Avg merge
2d 58m
Merged PRs (30d)
233

Description

See https://checkstyle.org/config_misc.html#Indentation

Currently, it's `lineWrappingIndentation` and `forceStrictCondition` are not set:
https://github.com/apache/incubator-druid/blob/ac856fe4c179093c0a5e309a2e988c54a86e89c5/codestyle/checkstyle.xml#L112-L115

Contributor guide

Open the contributing guide

Research direction

Start with codestyle/checkstyle.xml around lines 112-115 and read the linked Checkstyle Indentation documentation. Determine the intended settings for lineWrappingIndentation and forceStrictCondition, then verify that both options are explicitly configured in the checkstyle file.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
tooling
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.