Skip to content
This repository was archived by the owner on Sep 6, 2018. It is now read-only.

Commit 436d6ee

Browse files
authored
Merge #32: Fix swiftlint errors on enum names
Fix swiftlint errors on enum names not conforming to the Swift 3 syntax.
2 parents d8421d3 + c4e8938 commit 436d6ee

File tree

4 files changed

+27
-25
lines changed

4 files changed

+27
-25
lines changed

CHANGELOG.md

+3-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,9 @@ _None_
1818

1919
### Internal Changes
2020

21-
_None_
21+
* Fix swiftlint errors on enum names not conforming to the Swift 3 syntax.
22+
[Liquidsoul](https://github.com/liquidsoul)
23+
[#31](https://github.com/SwiftGen/SwiftGenKit/issues/31)
2224

2325
## 1.1.0
2426

Sources/Parsers/ColorsFileParser.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public enum ColorsParserError: Error, CustomStringConvertible {
1919
public var description: String {
2020
switch self {
2121
case .invalidHexColor(string: let string, key: let key):
22-
let keyInfo = key.flatMap { k in " for key \"\(k)\"" } ?? ""
22+
let keyInfo = key.flatMap { " for key \"\($0)\"" } ?? ""
2323
return "error: Invalid hex color \"\(string)\" found\(keyInfo)."
2424
case .invalidFile(reason: let reason):
2525
return "error: Unable to parse file. \(reason)"

Sources/Parsers/StringsFileParser.swift

+14-14
Original file line numberDiff line numberDiff line change
@@ -51,31 +51,31 @@ public final class StringsFileParser {
5151
// MARK: - Public Enum types
5252

5353
public enum PlaceholderType: String {
54-
case Object = "String"
55-
case Float = "Float"
56-
case Int = "Int"
57-
case Char = "Character"
58-
case CString = "UnsafePointer<unichar>"
59-
case Pointer = "UnsafePointer<Void>"
60-
case Unknown = "UnsafePointer<()>"
54+
case object = "String"
55+
case float = "Float"
56+
case int = "Int"
57+
case char = "Character"
58+
case cString = "UnsafePointer<unichar>"
59+
case pointer = "UnsafePointer<Void>"
60+
case unknown = "UnsafePointer<()>"
6161

6262
init?(formatChar char: Character) {
6363
guard let lcChar = String(char).lowercased().characters.first else {
6464
return nil
6565
}
6666
switch lcChar {
6767
case "@":
68-
self = .Object
68+
self = .object
6969
case "a", "e", "f", "g":
70-
self = .Float
70+
self = .float
7171
case "d", "i", "o", "u", "x":
72-
self = .Int
72+
self = .int
7373
case "c":
74-
self = .Char
74+
self = .char
7575
case "s":
76-
self = .CString
76+
self = .cString
7777
case "p":
78-
self = .Pointer
78+
self = .pointer
7979
default:
8080
return nil
8181
}
@@ -176,7 +176,7 @@ public final class StringsFileParser {
176176
}
177177
if insertionPos > 0 {
178178
while list.count <= insertionPos-1 {
179-
list.append(.Unknown)
179+
list.append(.unknown)
180180
}
181181
list[insertionPos-1] = p
182182
}

Tests/SwiftGenKitTests/StringsTests.swift

+9-9
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class StringsTests: XCTestCase {
2828
translation: "My awesome title"))
2929
parser.addEntry(StringsFileParser.Entry(key: "Greetings",
3030
translation: "Hello, my name is %@ and I'm %d",
31-
types: .Object, .Int))
31+
types: .object, .int))
3232

3333
let result = parser.stencilContext()
3434
XCTDiffContexts(result, expected: "entries.plist", sub: .strings)
@@ -78,44 +78,44 @@ class StringsTests: XCTestCase {
7878

7979
func testParseStringPlaceholder() {
8080
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%@")
81-
XCTAssertEqual(placeholders, [.Object])
81+
XCTAssertEqual(placeholders, [.object])
8282
}
8383

8484
func testParseFloatPlaceholder() {
8585
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%f")
86-
XCTAssertEqual(placeholders, [.Float])
86+
XCTAssertEqual(placeholders, [.float])
8787
}
8888

8989
func testParseDoublePlaceholders() {
9090
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%g-%e")
91-
XCTAssertEqual(placeholders, [.Float, .Float])
91+
XCTAssertEqual(placeholders, [.float, .float])
9292
}
9393

9494
func testParseFloatWithPrecisionPlaceholders() {
9595
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%1.2f : %.3f : %+3f : %-6.2f")
96-
XCTAssertEqual(placeholders, [.Float, .Float, .Float, .Float])
96+
XCTAssertEqual(placeholders, [.float, .float, .float, .float])
9797
}
9898

9999
func testParseIntPlaceholders() {
100100
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%d-%i-%o-%u-%x")
101-
XCTAssertEqual(placeholders, [.Int, .Int, .Int, .Int, .Int])
101+
XCTAssertEqual(placeholders, [.int, .int, .int, .int, .int])
102102
}
103103

104104
func testParseCCharAndStringPlaceholders() {
105105
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%c-%s")
106-
XCTAssertEqual(placeholders, [.Char, .CString])
106+
XCTAssertEqual(placeholders, [.char, .cString])
107107
}
108108

109109
func testParsePositionalPlaceholders() {
110110
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: "%2$d-%4$f-%3$@-%c")
111-
XCTAssertEqual(placeholders, [.Char, .Int, .Object, .Float])
111+
XCTAssertEqual(placeholders, [.char, .int, .object, .float])
112112
}
113113

114114
func testParseComplexFormatPlaceholders() {
115115
let format = "%2$1.3d - %4$-.7f - %3$@ - %% - %5$+3c - %%"
116116
let placeholders = StringsFileParser.PlaceholderType.placeholders(fromFormat: format)
117117
// positions 2, 4, 3, 5 set to Int, Float, Object, Char, and position 1 not matched, defaulting to Unknown
118-
XCTAssertEqual(placeholders, [.Unknown, .Int, .Object, .Float, .Char])
118+
XCTAssertEqual(placeholders, [.unknown, .int, .object, .float, .char])
119119
}
120120

121121
func testParseEscapePercentSign() {

0 commit comments

Comments
 (0)