QuantStack / QuantStack/git2cpp
Handle `cerr` in terminal pager
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 6
- Forks
- 5
- Ø Merge
- 4 Std. 22 Min.
- Gemergte PRs (30 T.)
- 4
Beschreibung
Need to handle cerr as well as cout in terminal pager. Discussion from #46:
Ian:
We do need a strategy for what to do with cerr, but I don't know what is best yet.
Currently cerr will be written to the normal terminal buffer, so it won't be seen whilst the alternative buffer is being used but it reappears when the alternative buffer is disabled. This is perhaps not a good solution, but I don't think it is too bad for a first implementation. Some other options:
- Capture it and sent it to the same place as cout, it will appear in the output flow in the alternative buffer. We'd probably need to colour it to make is easily visible as it could be anywhere.
- Capture it and display it at the bottom of the alternative buffer for better visibility, but then there might be quite a lot of information displayed.
- Capture it and if anything at all is written to it we could avoid using the pager at all, just display the errors in the normal terminal flow.
Johan:
I have a preference for option 1, but I think we can discuss it in a dedicated issue and solve it in a dedicated PR so that it does not block this one, WDYT?
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
Das Issue nennt keine Dateien, Tests oder einen spezifischen Einstiegspunkt. Beginne damit, die Implementierung des Terminal-Pagers zu finden und die Diskussion aus #46 zu lesen, und stelle dann fest, welche Strategie für den Umgang mit cerr das Projekt akzeptiert. Erledigt ist die Aufgabe, wenn cerr entsprechend dieser Entscheidung behandelt wird, ohne die Arbeit am Pager zu blockieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- cli
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 30/100