-
Notifications
You must be signed in to change notification settings - Fork 7
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
Support .config directory #180
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Darn it. The classic "I have an AUR package installed and didn't realize" -case. Let me try again 😄 |
Alright I gave it a good try with each supported path and works great + I like the flexibility of it The only thing is that if I have an empty What do you think about that? |
Currently, the first path that has a valid JSON object will be used. An empty file is not valid JSON. We can change it to try using the first path with a file no matter what. Do you think that's better? |
Hmm. Well, that would match my expectation of what should happen, but is that a small and easy change? |
Yes haha: bc70a8c. Can you verify that it works how you would expect now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lol it's the smallest change of all time 🎉
Yes I think it's quite perfect now, thanks!
Close #179