-
Notifications
You must be signed in to change notification settings - Fork 442
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
Update config path to comply with XDG Base Directory spec #90
base: master
Are you sure you want to change the base?
Conversation
I am not sure what else files are used, some help would be appreciated! |
Current coverage is 95.18% (diff: 100%)@@ master #90 diff @@
==========================================
Files 34 34
Lines 2076 2076
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 1976 1976
Misses 100 100
Partials 0 0
|
Hi @SanketDG, you're on a roll, thanks for the second PR! I'll try to review it this week. |
Hi, took a closer look... From https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html:
On my system, I don't have $XDG_CONFIG_HOME configured. It seems like we'd have to add additional logic to set the path to Also, curious how this affects Windows. |
|
|
|
|
https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html
Issue: #13