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

Added --print-path flag to print common paths #4307

Closed
wants to merge 1 commit into from

Conversation

mitsuhiko
Copy link
Contributor

I was trying to debug why stuff was not loading on my windows machine and learned in the process that the paths are quite different. Since I was trying to rewrite the config file for a test automatically I needed to add platform specific logic. That might be something people need to do more commonly so this adds a --print-path argument that can be used for scripting.

Example:

$ hx --print-path config-file
/Users/mitsuhiko/.config/helix/config.toml

Feel free to close this down. This seems valuable to me, but it might be too specific.

@the-mikedavis
Copy link
Member

These should be printed by hx --health and not add a new flag

@mitsuhiko
Copy link
Contributor Author

My use for this was external scripting. Parsing the output of hx --health seems brittle.

@kirawi kirawi added A-helix-term Area: Helix term improvements S-waiting-on-review Status: Awaiting review from a maintainer. labels Oct 25, 2022
@gibbz00 gibbz00 mentioned this pull request Feb 22, 2023
5 tasks
@pascalkuthe
Copy link
Member

I am closing this one out as stale. We want to keep the command line very minimal. These days health prints most of these flags. I don't see us adding a dedicated flag. Thank you for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants