feature(args): Add clean mode

felipe-heredia commented 1 month ago

Description

A description of the change, what it does, and why it was made. If relevant (such as any change that modifies the UI), please provide screenshots of the changes:

This PR introduces a new argument, clean, that adds a clean mode to the application.

Issue

This PR is directly related to #31 issue.

Closes: #31

Testing

This was tested passing --clean as an argument.

If this is a code change, please also indicate which platforms were tested:

[ ] Windows
[ ] macOS
[x] Linux

Checklist

If relevant, ensure the following have been met:

[x] Areas your change affects have been linted using rustfmt (cargo fmt)
[x] The change has been tested and doesn't appear to cause any unintended breakage
[x] Documentation has been added/updated if needed (README.md, help menu, doc pages, etc.)
[x] The pull request passes the provided CI pipeline
[x] There are no merge conflicts
[x] If relevant, new tests were added (don't worry too much about coverage)

ClementTsang commented 1 month ago

Thanks for the PR! This has been a long standing issue, so I'm glad to see someone grab it.

That said, I'm not totally a fan of adding a CLI flag specifically for it atm. This feels more like something I would set (IMO) as some style setting in a config/options screen. I might merge this for now but maybe change that specific part in the future.

felipe-heredia commented 1 month ago

Thanks for your comment and for a possible merge.

Yeah, that's a better idea, setting a config/option screen.

This was my first contribution to Rust projects, so I've tried my best at the moment, but your idea is so much better option.

ClementTsang / bottom