stackabletech / stackabletech/stackablectl
feat: Implement colored output (with `NO_COLOR` support)
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Rust
- Sterne
- 10
- Forks
- 5
- Ø Merge
- 4 Std. 41 Min.
- Gemergte PRs (30 T.)
- 4
Beschreibung
Colours sould only be enabled when the terminal supports them.
For example, when redirecting output or piping to cat, there should be no escape sequences.
I believe there needs to be a condition around the pretty method (or maybe it should be handled by tracing_subscriber::fmt):
Examples:
stackablectl --log-level=DEBUG stack list > /tmp/stackablectl.stdout # plain redirect
stackablectl --log-level=DEBUG stack list | cat > /tmp/stackablectl.stdout # pipe through cat
TERM= stackablectl --log-level=DEBUG stack list > /tmp/stackablectl.stdout # explicitly set TERM to nothing
If I open it in VIM, it looks like this:
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in rust/stackablectl/src/main.rs rund um die verlinkte pretty-Methode und vergleiche, ob tracing_subscriber::fmt die Terminalerkennung übernehmen sollte. Führe die aufgelisteten stackablectl-Befehle mit umgeleiteter und weitergeleiteter Ausgabe aus, einschließlich eines leeren TERM, und bestätige, dass nicht-interaktive Ausgabe keine Escape-Sequenzen enthält, während die Ausgabe unterstützter Terminals weiterhin farbig bleibt und NO_COLOR berücksichtigt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- cli
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 50/100