-
Notifications
You must be signed in to change notification settings - Fork 471
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
Add “doctor” chezmoi command #137
Conversation
This PR is related to the |
Oops, I had an error in the paths being printed. Fixed in d585022:
|
If you prefer aligned, we can use a tabwriter (https://golang.org/pkg/text/tabwriter/) to align it like so:
|
#138 is also relevant now :) |
Output:
|
For the output, let's focus on the logic first and then make the UX pretty second :) |
Updated output:
|
Another thing to add: please return |
What situations do you define as "not ok"? |
Good question! I would say anything that prevents |
Added exit codes 👍 |
Updated output:
EDIT: Updated to show the editor component |
Thanks for addressing the commands. Please can you squash the commits in to a single one with the message "Add doctor command" and I will then merge. |
Well, squashing stuff broke the PR 😢 |
Add a
chezmoi doctor
command. Not sure the best way to test this though (unit test wise). Thoughts?