hyperoslo / hyperoslo/Cache

memoryConfig not optional for Storage.init as comment says.

Aperta
#330 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Swift
Stelle
3.1k
Fork
355
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

``` /// 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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Review the public Storage.init(diskConfig:memoryConfig:transformer:) declaration shown in the issue and inspect its callers to determine whether memoryConfig is intended to be optional. Make the signature and documentation consistent, then verify the initializer behavior and existing call sites.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
swift
Ambito
api
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.