Merge pull request #83 from WeTransfer/feature/fix-swiftlint-issues

Fix SwiftLint and SwiftFormat warnings

Author: AvdLee

Package.swift

@@ -6,7 +6,7 @@ let package = Package(
     name: "GitBuddy",
     platforms: [
         .macOS(.v10_15)
-        ],
+    ],
     products: [
         .executable(name: "GitBuddy", targets: ["GitBuddy"])
     ],

Sources/GitBuddyCore/Changelog/ChangelogItemsFactory.swift

@@ -21,7 +21,7 @@ struct ChangelogItemsFactory {
                 return [ChangelogItem(input: pullRequest, closedBy: pullRequest)]
             }
             return resolvedIssues.map { issue -> ChangelogItem in
-                return ChangelogItem(input: issue, closedBy: pullRequest)
+                ChangelogItem(input: issue, closedBy: pullRequest)
             }
         }
     }

Sources/GitBuddyCore/Changelog/ChangelogProducer.swift

@@ -11,14 +11,14 @@ import OctoKit
 
 /// Capable of producing a changelog based on input parameters.
 final class ChangelogProducer: URLSessionInjectable {
-
     enum Since {
         case date(date: Date)
         case tag(tag: String)
         case latestTag
 
         /// Gets the date for the current Since property.
-        /// In the case of a tag, we add 60 seconds to make sure that the Changelog does not include the commit that is used for creating the tag.
+        /// In the case of a tag, we add 60 seconds to make sure that the Changelog does not include the commit that
+        /// is used for creating the tag.
         /// This is needed as the tag creation date equals the commit creation date.
         func get() throws -> Date {
             switch self {
@@ -32,7 +32,7 @@ final class ChangelogProducer: URLSessionInjectable {
         }
     }
 
-    private lazy var octoKit: Octokit = Octokit()
+    private lazy var octoKit: Octokit = .init()
     let baseBranch: Branch
     let since: Since
     let from: Date

Sources/GitBuddyCore/Commands/ChangelogCommand.swift

@@ -5,11 +5,10 @@
 //  Created by Antoine van der Lee on 09/04/2020.
 //
 
-import Foundation
 import ArgumentParser
+import Foundation
 
 struct ChangelogCommand: ParsableCommand {
-
     public static let configuration = CommandConfiguration(commandName: "changelog", abstract: "Create a changelog for GitHub repositories")
 
     @Option(name: .shortAndLong, help: "The tag to use as a base. Defaults to the latest tag.")

Sources/GitBuddyCore/Commands/GitBuddy.swift

@@ -6,8 +6,8 @@
 //  Copyright © 2020 WeTransfer. All rights reserved.
 //
 
-import Foundation
 import ArgumentParser
+import Foundation
 
 /// Entry class of GitBuddy that registers commands and handles execution.
 public struct GitBuddy: ParsableCommand {
@@ -20,5 +20,5 @@ public struct GitBuddy: ParsableCommand {
         subcommands: [ChangelogCommand.self, ReleaseCommand.self, TagDeletionsCommand.self]
     )
 
-    public init() { }
+    public init() {}
 }

Sources/GitBuddyCore/Commands/ReleaseCommand.swift

@@ -5,12 +5,14 @@
 //  Created by Antoine van der Lee on 09/04/2020.
 //
 
-import Foundation
 import ArgumentParser
+import Foundation
 
 struct ReleaseCommand: ParsableCommand {
-
-    public static let configuration = CommandConfiguration(commandName: "release", abstract: "Create a new release including a changelog and publish comments on related issues.")
+    public static let configuration = CommandConfiguration(
+        commandName: "release",
+        abstract: "Create a new release including a changelog and publish comments on related issues."
+    )
 
     @Option(name: .shortAndLong, help: "The path to the Changelog to update it with the latest changes.")
     var changelogPath: String?
@@ -22,11 +24,26 @@ struct ReleaseCommand: ParsableCommand {
     var isPrerelease: Bool = false
 
     // Main
-    @Option(name: .shortAndLong, help: "Specifies the commitish value that determines where the Git tag is created from. Can be any branch or commit SHA. Unused if the Git tag already exists. Default: the repository's default branch (usually main).")
+    @Option(
+        name: .shortAndLong,
+        help: """
+            Specifies the commitish value that determines where the Git tag is created from. Can be any branch or commit SHA.
+            Unused if the Git tag already exists.
+
+            Default: the repository's default branch (usually main).
+        """
+    )
     var targetCommitish: String?
 
     // 1.3.1b1072
-    @Option(name: [.long, .customShort("n")], help: "The name of the tag. If set, `changelogToTag` is required too. Default: takes the last created tag to publish as a GitHub release.")
+    @Option(
+        name: [.long, .customShort("n")],
+        help: """
+            The name of the tag. If set, `changelogToTag` is required too.
+
+            Default: takes the last created tag to publish as a GitHub release.
+        """
+    )
     var tagName: String?
 
     // // 1.3.1b1072 - App Store Release
@@ -37,7 +54,15 @@ struct ReleaseCommand: ParsableCommand {
     @Option(name: .shortAndLong, help: "The last release tag to use as a base for the changelog creation. Default: previous tag.")
     var lastReleaseTag: String?
 
-    @Option(name: .customLong("changelogToTag"), help: "If set, the date of this tag will be used as the limit for the changelog creation. This variable should be passed when `tagName` is set. Default: latest tag.")
+    @Option(
+        name: .customLong("changelogToTag"),
+        help: """
+            If set, the date of this tag will be used as the limit for the changelog creation.
+            This variable should be passed when `tagName` is set.
+
+            Default: latest tag.
+            """
+    )
     var changelogToTag: String?
 
     // Develop

Sources/GitBuddyCore/Commands/TagDeletionsCommand.swift

@@ -1,9 +1,11 @@
-import Foundation
 import ArgumentParser
+import Foundation
 
 struct TagDeletionsCommand: ParsableCommand {
-
-    public static let configuration = CommandConfiguration(commandName: "tagDeletion", abstract: "Delete a batch of tags based on given predicates.")
+    public static let configuration = CommandConfiguration(
+        commandName: "tagDeletion",
+        abstract: "Delete a batch of tags based on given predicates."
+    )
 
     @Option(name: .shortAndLong, help: "The date of this tag will be used as a limit. Defaults to the latest tag.")
     private var upUntilTag: String?

Sources/GitBuddyCore/GitHub/Commenter.swift

@@ -18,8 +18,10 @@ enum Comment {
         case .releasedPR(let release):
             return "Congratulations! :tada: This was released as part of [Release \(release.title)](\(release.url)) :rocket:"
         case .releasedIssue(let release, let pullRequestID):
-            var body = "The pull request #\(pullRequestID) that closed this issue was merged and released as part of [Release \(release.title)](\(release.url)) :rocket:\n"
-            body += "Please let us know if the functionality works as expected as a reply here. If it does not, please open a new issue. Thanks!"
+            var body = "The pull request #\(pullRequestID) that closed this issue was merged and released "
+            body += "as part of [Release \(release.title)](\(release.url)) :rocket:\n"
+            body += "Please let us know if the functionality works as expected as a reply here. "
+            body += "If it does not, please open a new issue. Thanks!"
             return body
         }
     }
@@ -43,14 +45,17 @@ enum Commenter {
     static func post(_ comment: Comment, on issueID: Int, at project: GITProject, completion: @escaping () -> Void) {
         var body = comment.body
         body += watermark
-        Octokit().commentIssue(urlSession, owner: project.organisation, repository: project.repository, number: issueID, body: body) { response in
-            switch response {
-            case .success(let comment):
-                Log.debug("Successfully posted comment at: \(comment.htmlURL)")
-            case .failure(let error):
-                Log.debug("Posting comment for issue #\(issueID) failed: \(error)")
+        Octokit()
+            .commentIssue(urlSession, owner: project.organisation, repository: project.repository, number: issueID,
+                          body: body)
+            { response in
+                switch response {
+                case .success(let comment):
+                    Log.debug("Successfully posted comment at: \(comment.htmlURL)")
+                case .failure(let error):
+                    Log.debug("Posting comment for issue #\(issueID) failed: \(error)")
+                }
+                completion()
             }
-            completion()
-        }
     }
 }

Sources/GitBuddyCore/GitHub/IssuesFetcher.swift

@@ -9,7 +9,6 @@ import Foundation
 import OctoKit
 
 struct IssuesFetcher {
-
     let octoKit: Octokit
     let project: GITProject
 
@@ -20,7 +19,7 @@ struct IssuesFetcher {
 
         var result: Result<[Issue], Swift.Error>!
 
-        octoKit.issues(session, owner: project.organisation, repository: project.repository, state: .closed) { (response) in
+        octoKit.issues(session, owner: project.organisation, repository: project.repository, state: .closed) { response in
             switch response {
             case .success(let issues):
                 result = .success(issues)
@@ -42,5 +41,4 @@ struct IssuesFetcher {
             return closedAt > fromDate && closedAt < toDate
         }
     }
-
 }

Sources/GitBuddyCore/GitHub/IssuesResolver.swift

@@ -42,7 +42,6 @@ struct IssuesResolver {
 }
 
 extension String {
-
     /// Extracts the resolved issues from a Pull Request body.
     func resolvingIssues() -> [Int] {
         var resolvedIssues = Set<Int>()
@@ -69,8 +68,8 @@ extension String {
                     guard index + 1 <= splits.count - 1 else { break }
                     let nextSplit = splits[index + 1]
 
-                    let numberPrefixString = nextSplit.prefix { (character) -> Bool in
-                        return character.isNumber
+                    let numberPrefixString = nextSplit.prefix { character -> Bool in
+                        character.isNumber
                     }
 
                     if !numberPrefixString.isEmpty, let numberPrefix = Int(numberPrefixString.description) {