se-edu/addressbook-level2

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

Opened this issue · 1 comments

yamgent commented

Prerequisite: se-edu/addressbook-level4#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

damithc commented

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

  • 👍1