hyperoslo / hyperoslo/Cache

memoryConfig not optional for Storage.init as comment says.

Offen
#330 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Swift
Sterne
3.1k
Forks
355
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

``` /// Initialize storage with configuration options.
///
/// - Parameters:
/// - diskConfig: Configuration for disk storage
/// - memoryConfig: *Optional*. Pass config if you want memory cache
/// - Throws: Throw StorageError if any.
public convenience init(diskConfig: DiskConfig, memoryConfig: MemoryConfig, transformer: Transformer) throws {
let disk = try DiskStorage(config: diskConfig, transformer: transformer)
let memory = MemoryStorage(config: memoryConfig)
let hybridStorage = HybridStorage(memoryStorage: memory, diskStorage: disk)
self.init(hybridStorage: hybridStorage)
}
```

This says `memoryConfig` should be Optional, but it's not. Not sure which is the typo.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Überprüfe die im Issue gezeigte öffentliche Storage.init(diskConfig:memoryConfig:transformer:)-Deklaration und untersuche ihre Aufrufer, um festzustellen, ob memoryConfig optional sein soll. Mache die Signatur und die Dokumentation konsistent und überprüfe anschließend das Verhalten des Initializers und die vorhandenen Aufrufstellen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
swift
Bereich
api
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.