New Hire on LDD Team Removes Over 500 Code Convention Violations Using Terminal Tools

Lee Jong-rip, a new hire on the LDD team, spent his first week at the company performing automated work using terminal tools to comply with the team's coding style guide. By combining commands such as find, egrep, ag, and sed, he found and fixed over 500 cases of convention violations, including omitted braces, missing whitespace, and use of tab characters.

Incremental Improvement Strategy

Instead of applying a formatter across the entire codebase at once, he adopted the Boy Scout Rule, cleaning up a little each time an issue was found. This was intended as an opportunity for the new hire to learn Git, terminal commands, and the Pull Request workflow. Of the 1,823 warnings generated by running CheckStyle, a significant portion were removed, and the remaining warnings are planned to be addressed incrementally.

Key Fixes

The specific work performed was as follows:

• No omitted braces: Fixed 5 violations where braces were omitted in if statements and the like.
• Added whitespace: Added spaces after if, for, while, and try keywords before parentheses, and added spaces on both sides of the arrow operator (->).
• Fixed variable names: Changed lambda variable names starting with uppercase letters to lowercase.
• Replaced tab characters: Batch-replaced tab characters in .java files with 2 spaces.

All of this work was efficiently carried out using the sed and ag tools, and each change was merged via a Pull Request. As a result, the project's code convention compliance rate improved, and it contributed to establishing careful Git commit message writing and a code review process among team members.