lightmon icon indicating copy to clipboard operation
lightmon copied to clipboard

Big refactor and various improvements

Open reaganmcf opened this issue 3 years ago • 2 comments

Since I haven't worked on this project in a while, I have learned a lot about Rust in the meantime. Going back over this codebase I see lots of bad practices and better ways to do things.

However, I'm not changing any variable names yet or any architectural changes, just refactoring where it is trivial. Also, I am removing logging since we don't need anything that intense, and print statements are more than enough for this application.

Lastly, this patch also is meant to close #1, since I did implement colored output and better output messages. Below is a screenshot Screen Shot 2021-07-19 at 10 52 27 PM

reaganmcf avatar Jul 20 '21 02:07 reaganmcf

@alayshahh No idea why the windows stuff isn't working - going to fix that later. But, if you could review the rest of the patch that would be appreciated 👍🏻

reaganmcf avatar Jul 22 '21 01:07 reaganmcf

I'm gonna take a look through https://github.com/nushell/nushell to see how they do it. They support all platforms and get colored output with no issues, by maintaining a modified version of ansi_term called nu_ansi_term

reaganmcf avatar Jul 29 '21 21:07 reaganmcf