0xMiden / 0xMiden/rust-sdk

refactor: reduce calls for `load_config_file` in CLI

Offen
#857 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
78
Forks
129
Ø Merge
4 T. 14 Std.
Gemergte PRs (30 T.)
52

Beschreibung

> I wonder if we can instantiate the `CliConfig` within the function to avoid passing it from `main()` essentially. But maybe this is better to avoid loading the file many times (maybe it could be within a once cell?). There are other occurrences of this as well.

_Originally posted by @igamigo in https://github.com/0xPolygonMiden/miden-client/pull/840#discussion_r2047501343_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Schauen Sie sich den CLI-Code an, wahrscheinlich in einer main.rs- oder cli.rs-Datei, um die Funktion `load_config_file` und die Struktur `CliConfig` zu finden. Das Ziel ist es, wiederholtes Laden von Dateien zu reduzieren, möglicherweise durch Verwendung eines Once-Cell- oder Caching-Mechanismus. Untersuchen Sie, wie die Konfiguration derzeit von `main()` übergeben wird, und identifizieren Sie alle Aufrufstellen. Eine erfolgreiche Änderung lädt die Konfiguration einmal und verwendet sie wieder.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
cli
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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