refactor(config): simplify config path finding - Githubissues

orhun / systeroid

A more powerful alternative to sysctl(8) with a terminal user interface 🐧

https://systeroid.cli.rs

Apache License 2.0

1.28k stars 23 forks source link

refactor(config): simplify config path finding #145

Closed matthiasbeyer closed 1 year ago

matthiasbeyer commented 1 year ago

Motivation and Context

I just found that this could be written with less code, so that's what I did here.

How Has This Been Tested?

cargo test

app::tests::test_app fails for me, but it also does on latest main for me, so I figured I can file this.

Screenshots / Logs (if applicable)

Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[x] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.