GitHub/SwiftLint
1
0
Fork 0
mirror of https://github.com/realm/SwiftLint.git synced 2026-06-06 20:18:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

Uses correct enumcase instead of enum

Browse Source
This commit is contained in:
Kane Cheshire
2021-05-07 11:04:20 +01:00
committed by Marcelo Fabri
parent 538764278b
commit 0dd96d249e
1 changed files with 3 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Source/SwiftLintFramework/Rules/Idiomatic/DisallowNoneCase.swift
+3 -9
View File
@@ -10,13 +10,7 @@ public struct DisallowNoneCase: ASTRule, OptInRule, ConfigurationProviderRule {
identifier: "disallow_none_case",
name: "Disallow None Case",
description: "Disallows the naming of enum cases as 'none' which can conflict with Optional<T>.none",
kind: .idiomatic,
nonTriggeringExamples: [
Example("")
],
triggeringExamples: [
Example("")
]
kind: .idiomatic
)
public func validate(
@@ -31,7 +25,7 @@ public struct DisallowNoneCase: ASTRule, OptInRule, ConfigurationProviderRule {
severity: configuration.severity,
location: Location(file: file, byteOffset: offset),
reason: """
\(kind.reasonPrefix) should not be named `none` since the compiler can think you mean `Optional<T>.none`.
\(kind.reasonPrefix) should not be named `none` since the compiler can think you mean `Optional<T>.none when checking equality`.
"""
)
]
@@ -39,7 +33,7 @@ public struct DisallowNoneCase: ASTRule, OptInRule, ConfigurationProviderRule {
}
private extension SwiftDeclarationKind {
var isForValidating: Bool { self == .enum || self == .varClass || self == .varStatic }
var isForValidating: Bool { self == .enumcase || self == .varClass || self == .varStatic }
var reasonPrefix: String {
switch self {