Handle hyphens internally
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 42/100
Rechercherichtung
Beginne mit der in der Issue erwähnten Funktion parse_spec() und prüfe den bestehenden Dokumentationshinweis aus PR #77 zu Bindestrichen in crate-Namen. Überprüfe das in Issue #190 referenzierte Build-Problem, bevor du die Änderung untersuchst. Erledigt ist es, wenn Bindestriche intern verarbeitet werden, sodass Benutzer sie nicht manuell ersetzen müssen, und das Logging wie erwartet funktioniert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
With PR #77 a note about dashes in the crate name was added in the docs. The note was of course an essential addition.
I propose going a step further and handling the dashes internally, during the parsing (parse_spec() function).
I believe this addition will save people time trying to debug the reason that the logging does not work as expected.
I myself lost some time on this, even though I had read the docs, and I had read about the need to replace the dashes.
I would also have provided a PR too, but currently I am failing to build the library (see https://github.com/env-logger-rs/env_logger/issues/190#issuecomment-812988592).
Is there a valid reason for not handling the dashes internally?
- Vorherrschende Sprache
- Rust
- Sterne
- 1.1k
- Forks
- 149
- Ø Merge
- 2 Min.
- Gemergte PRs (30 T.)
- 1
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus rust-cli/env_logger
-
Sanitization requires color, which parses every recordEvtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100
rust-cli/env_logger#419 · 2 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
rust-cli/env_logger#379 · 5 Kommentare ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
rust-cli/env_logger#369 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
rust-cli/env_logger#367 · 1 Kommentar ·
-
refactor the crate to separate the provided configurable formater and the optional custom formatterOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rust-cli/env_logger#365 · 10 Kommentare ·
Alle Issues in rust-cli/env_logger
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
compiler P4 perf
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
HerringtonDarkholme/hd-lang#1044 ·
Maintainer antworten meist innerhalb von 1 Tag
-
good first issue help wanted r-docs
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
servo/rust-smallvec#824 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
akitaonrails/ai-usagebar#552 ·
Maintainer antworten meist innerhalb von 1 Tag