Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(tui): use stdout for rendering #79

Merged
merged 1 commit into from
Oct 10, 2024
Merged

feat(tui): use stdout for rendering #79

merged 1 commit into from
Oct 10, 2024

Conversation

orhun
Copy link
Owner

@orhun orhun commented Oct 10, 2024

Description of change

Use stdout instead of stderr, see https://blog.orhun.dev/stdout-vs-stderr/

How has this been tested? (if applicable)

Locally.

Copy link

codecov bot commented Oct 10, 2024

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 3.34%. Comparing base (ffb35ea) to head (35d710b).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/tui/mod.rs 0.00% 4 Missing ⚠️
src/lib.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main     #79      +/-   ##
========================================
+ Coverage   3.33%   3.34%   +0.01%     
========================================
  Files         19      19              
  Lines       2407    2401       -6     
========================================
  Hits          80      80              
+ Misses      2327    2321       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@orhun orhun merged commit cb27463 into main Oct 10, 2024
27 of 28 checks passed
@orhun orhun deleted the chore/switch_to_stdout branch October 10, 2024 21:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant