-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Closed
Labels
good first issueIssue to be taken up by new contributors yet unfamiliar with the project.Issue to be taken up by new contributors yet unfamiliar with the project.rule-requestRequests for a new rules.Requests for a new rules.
Description
The modifiers final
and class
on a method/property have the same meaning as static
. The rule should flag appearances of final class
on the same declaration and replace them by static
.
I don't think there is any need for configuration. The rule could be enabled by default, but this can be decided in review when browsing through the reported OSS findings.
Triggering:
class C {
final class func f() {}
}
final class C {
class func f() {}
}
Non-triggering:
class C {
static func f() {}
}
class C {
class func f() {}
}
final class C {}
Metadata
Metadata
Assignees
Labels
good first issueIssue to be taken up by new contributors yet unfamiliar with the project.Issue to be taken up by new contributors yet unfamiliar with the project.rule-requestRequests for a new rules.Requests for a new rules.