Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Leftwm-check #191

Merged
merged 3 commits into from
Feb 17, 2021
Merged

Leftwm-check #191

merged 3 commits into from
Feb 17, 2021

Conversation

mautamu
Copy link
Member

@mautamu mautamu commented Feb 14, 2021

Good morning,

Per #168, and kind of #27, I think it would be helpful to have a binary that I have called cargo-check. As currently implemented, it will tell you if your configuration is right or give a TOML error if it is wrong. I think it could be expanded to:

  1. verify whether a theme is installed properly (verify theme config is correct)
  2. perhaps only be compiled as an optional feature for debugging purposes
  3. propose suggestions for fixing the config.toml file
  4. read config.toml files from other locations to check them as well (although Clap currently says it will take input, it won't)

Any thoughts? As it is written now, it is a bit messy in output. I found it helpful in developing a solution to #32 that I intend to finish soon.

Best,
Mautamu

@mautamu mautamu marked this pull request as ready for review February 17, 2021 07:23
@lex148 lex148 merged commit d4bce1c into leftwm:master Feb 17, 2021
mautamu added a commit to mautamu/leftwm that referenced this pull request Mar 18, 2021
* Add checker

* Remove need to edit version every time in binary files

* Cargo FMT
mautamu added a commit to mautamu/leftwm that referenced this pull request May 19, 2021
* Add checker

* Remove need to edit version every time in binary files

* Cargo FMT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants