memoryConfig not optional for Storage.init as comment says.
- 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
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