Uploaded image for project: 'REEF (Retired)'
  1. REEF (Retired)
  2. REEF-332 Pass checkstyle
  3. REEF-388

Fix violations of RedundantModifier checkstyle

Details

    • Sub-task
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 0.12
    • All
    • None

    Description

      RedundantModifier (http://checkstyle.sourceforge.net/config_modifier.html#RedundantModifier) check suggests to remove modifiers when they are redundant. We have ~420 violations in our code, two most typical scenarios are:

      • public modifier on interface method
      • final modifier on method of final class

      If we intend to keep as close to existing checkstyle configuration as possible, we need to remove these modifiers (thus sacrificing a bit of readability).

      Attachments

        Activity

          People

            MariiaMykhailova Mariia Mykhailova
            MariiaMykhailova Mariia Mykhailova
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment