Skip to main content

Added missing @Override parameter (Sonar)

sonar:java/add-missing-override-s1161

ImportanceReview GuidanceRequires Scanning Tool
LOWMerge Without ReviewYes (Sonar)

This change adds missing @Override to known subclasses. Documenting inheritance will help readers and static analysis tools understand the code better, spot bugs easier, and in general lead to more efficient and effective review.

Our changes look something like this:

  interface AcmeParent {
void doThing();
}

class AcmeChild implements AcmeParent {

+ @Override
void doThing() {
thing();
}

}

F.A.Q.

Why is this codemod marked as Merge Without Review?

There is no functional difference after the change, but the source code will be easier to understand.

References