addressbook-level2 icon indicating copy to clipboard operation
addressbook-level2 copied to clipboard

Remove 'Check' suffix for module names in checkstyle.xml

Open yamgent opened this issue 8 years ago • 1 comments

Prerequisite: https://github.com/se-edu/addressbook-level4/pull/279 (further study about changes in checkstyle.xml is required)

Some module names in checkstyle.xml have the Check suffix. For example, <module name="TypeNameCheck">.

We should change them to be <module name="TypeName">, as the Check suffix is not needed for checkstyle to work. The module name would also be more concise if the suffix is removed.

Reference: http://stackoverflow.com/questions/41931046/checkstyle-module-name-constantname-vs-constantnamecheck

yamgent avatar Jan 31 '17 03:01 yamgent

@yamgent You may want to link to the ongoing PR at level-4 as this issue needs to wait until that PR is done.

damithc avatar Jan 31 '17 04:01 damithc