search

ndd7xv / heh

A terminal UI to edit bytes by the nibble.
MIT License
444 stars 18 forks source link

move from tui to ratatui #51

Closed ndd7xv closed 1 year ago

ndd7xv commented 1 year ago

see https://github.com/fdehau/tui-rs/issues/654 for more info

this also rebuilds the lock file; running a cargo tree -d --target all revealed a variety of dependencies that were being brought in with different versions, which now should not happen

this means smaller binary size (913408->909312 for windows) and less dependencies are built for heh to run (71->65 for windows, 68->66 for my local build)

this release of ratatui also improved the performance of split, so technically things should be slightly more performant lol

codecov[bot] commented 1 year ago

Codecov Report

Merging #51 (e00f549) into main (7fbccf6) will increase coverage by 5.3%. The diff coverage is 100.0%.

Additional details and impacted files | [Impacted Files](https://codecov.io/gh/ndd7xv/heh/pull/51?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [src/screen.rs](https://codecov.io/gh/ndd7xv/heh/pull/51?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3NjcmVlbi5ycw==) | `22.8% <100.0%> (+22.8%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://codecov.io/gh/ndd7xv/heh/pull/51/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)