gitsome icon indicating copy to clipboard operation
gitsome copied to clipboard

Update config path to comply with XDG Base Directory spec

Open SanketDG opened this issue 7 years ago • 9 comments

https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

Issue: https://github.com/donnemartin/gitsome/issues/13

SanketDG avatar Oct 31 '16 14:10 SanketDG

I am not sure what else files are used, some help would be appreciated!

SanketDG avatar Oct 31 '16 14:10 SanketDG

Current coverage is 95.18% (diff: 100%)

Merging #90 into master will not change coverage

@@             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          

Powered by Codecov. Last update eebb2cf...b9994bd

codecov-io avatar Oct 31 '16 14:10 codecov-io

Hi @SanketDG, you're on a roll, thanks for the second PR! I'll try to review it this week.

donnemartin avatar Nov 01 '16 01:11 donnemartin

Hi, took a closer look...

From https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html:

If $XDG_CONFIG_HOME is either not set or empty, a default equal to $HOME/.config should be used.

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 $HOME/.config?

Also, curious how this affects Windows.

donnemartin avatar Nov 06 '16 04:11 donnemartin

Die aktuelle Abdeckungbeträgt 95,18% (diff: 100%)

Das Zusammenführen von #90inMasterändert die Abdeckung nicht

@@             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          

Unterstützt vonCodecov. Letzte Aktualisierungeebb2cf... B9994BD

Jimimaku avatar Nov 21 '22 23:11 Jimimaku

Hallo, habe genauer hingeschaut...

Vonhttps://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html:

Wenn $XDG_CONFIG_HOME entweder nicht festgelegt oder leer ist, sollte der Standardwert $HOME/.config verwendet werden.

Auf meinem System habe ich $XDG_CONFIG_HOME nicht konfiguriert. Es scheint, als müssten wir zusätzliche Logik hinzufügen, um den Pfad festzulegen?$HOME/.config

Auch neugierig, wie sich dies auf Windows auswirkt.

Jimimaku avatar Nov 22 '22 00:11 Jimimaku

Hallo@SanketDG, du bist auf einer Rolle, danke für die zweite PR! Ich werde versuchen, es diese Woche zu überprüfen.

Jimimaku avatar Nov 22 '22 00:11 Jimimaku

Ich bin mir nicht sicher, welche anderen Dateien verwendet werden, etwas Hilfe wäre willkommen!

Jimimaku avatar Nov 22 '22 00:11 Jimimaku

https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

Ausgabe:#13

Jimimaku avatar Nov 22 '22 00:11 Jimimaku