kebab-case options

Author: swhitty

README.md

@@ -95,17 +95,18 @@ Options:
  --precision   maximum number of decimal places
  --output      optional path of output file
 
- --hideUnsupportedFilters   hide elements with unsupported filters.
+ --hide-unsupported-filters   hide elements with unsupported filters.
 
 Available keys for --format swift:
  --api                api of generated code:  appkit | uikit
 
 Available keys for --format sfsymbol:
  --insets             alignment of regular variant: top,left,bottom,right | auto
  --ultralight         svg file of ultralight variant
- --ultralightInsets   alignment of ultralight variant: top,left,bottom,right | auto
+ --ultralight-insets  alignment of ultralight variant: top,left,bottom,right | auto
  --black              svg file of black variant
- --blackInsets        alignment of black variant: top,left,bottom,right | auto
+ --black-insets       alignment of black variant: top,left,bottom,right | auto
+ --legacy             use the original, less precise alignment logic from earlier swiftdraw versions.
 ```
 
 ```bash

SwiftDraw/Sources/CommandLine/CommandLine.Configuration.swift

@@ -210,11 +210,14 @@ extension CommandLine {
 
     static func parseInsets(from value: String??) throws -> Insets? {
         guard let value = value,
-              let value = value,
-              value != "auto" else {
+              let value = value else {
             return nil
         }
 
+        guard value != "auto" else {
+            return Insets()
+        }
+
         var scanner = XMLParser.Scanner(text: value)
         let top = try scanner.scanInset()
         _ = try scanner.scanString(",")

SwiftDraw/Tests/CommandLine/CommandLine.ArgumentsTests.swift

@@ -42,7 +42,7 @@ final class CommandLineArgumentsTests: XCTestCase {
         XCTAssertEqual(modifiers, [.ultralightInsets: "a", .blackInsets: "b", .hideUnsupportedFilters: nil, .legacy: nil])
 
         modifiers = try CommandLine.parseModifiers(from: ["--ultralight-insets", "a", "--black-insets", "b", "--hide-unsupported-filters", "--legacy"])
-        XCTAssertEqual(modifiers, [.ultralightInsets: "a", .blackInsets: "b", .hideUnsupportedFilters: nil, .legacy: nil])"
+        XCTAssertEqual(modifiers, [.ultralightInsets: "a", .blackInsets: "b", .hideUnsupportedFilters: nil, .legacy: nil])
 
     }
 

SwiftDraw/Tests/CommandLine/CommandLine.ConfigurationTests.swift

@@ -90,9 +90,8 @@ final class CommandLineConfigurationTests: XCTestCase {
     }
 
     func testParseInsets() throws {
-        XCTAssertEqual(
-            try CommandLine.parseInsets(from: nil),
-            .init()
+        XCTAssertNil(
+            try CommandLine.parseInsets(from: nil)
         )
         XCTAssertEqual(
             try CommandLine.parseInsets(from: "auto"),