cake-build/cake

Use OS-native directory separators in Path.FullPath

Open

#2.367 geöffnet am 25. Nov. 2018

Auf GitHub ansehen
 (4 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)C# (738 Forks)batch import
Help wantedImprovementUp-for-grabs

Repository-Metriken

Stars
 (3.742 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 1h) (25 gemergte PRs in 30 T)

Beschreibung

There are applications which do not understand the alternate directory separator; for example, https://github.com/cake-build/cake/issues/2337. There have also been multiple CLI tools I've used where I had to do .FullName.Replace('/', '\\').

Would you consider returning OS-native directory separators from Path.FullName? It will just work in all situations, and it will not stand out and look strange on Windows any longer even when it does work. (At times, I've manually edited MSBuild logs etc so people don't ask why I'm using weird paths.)

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung von Path.FullPath in der Cake Codebasis und stellen Sie sicher, dass das native Verzeichnistrennzeichen des Betriebssystems (Path.DirectorySeparatorChar) verwendet wird, anstatt eines fest codierten Schrägstrichs. Überprüfen Sie dann, ob alle Tests unter Windows bestanden werden.
Tech Stack
csharp
Domain
build system
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70