Skip to content

Commit

Permalink
Remove dependency on RuuviLocalization in RuuviDFU
Browse files Browse the repository at this point in the history
Motivation: for single used string for the invalid arg case its not necessary to link against RuuviLocalization
  • Loading branch information
rinat-enikeev committed Dec 16, 2023
1 parent b34937c commit a97697d
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 9 deletions.
7 changes: 0 additions & 7 deletions Packages/RuuviDFU/Sources/RuuviDFU/RuuviDFUError.swift
Original file line number Diff line number Diff line change
@@ -1,13 +1,6 @@
import Foundation
import RuuviLocalization

public struct RuuviDfuError: Error {
public static let invalidFirmwareFile = RuuviDfuError(
description: RuuviLocalization.RuuviDfuError.invalidFirmwareFile
)
public static let failedToConstructUUID = RuuviDfuError(
description: RuuviLocalization.RuuviDfuError.failedToConstructUUID
)
public let description: String
public init(description: String) {
self.description = description
Expand Down
3 changes: 2 additions & 1 deletion Packages/RuuviDFU/Sources/RuuviDFUImpl/DfuFlasher.swift
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@ class DfuFlasher: NSObject {
) -> AnyPublisher<FlashResponse, Error> {
guard let uuid = UUID(uuidString: uuid)
else {
return Fail<FlashResponse, Error>(error: RuuviDfuError.failedToConstructUUID).eraseToAnyPublisher()
assertionFailure("Invalid UUID")
return Fail<FlashResponse, Error>(error: RuuviDfuError(description: "Invalid UUID")).eraseToAnyPublisher()
}
let subject = PassthroughSubject<FlashResponse, Error>()
self.subject = subject
Expand Down
1 change: 0 additions & 1 deletion Packages/RuuviDFU/target.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,3 @@ targets:
name: DFU
dependencies:
- package: NordicDFU
- target: RuuviLocalization

0 comments on commit a97697d

Please sign in to comment.