cake-build/cake

Use OS-native directory separators in Path.FullPath

Open

#2 367 ouverte le 25 nov. 2018

Voir sur GitHub
 (4 commentaires) (4 réactions) (0 assignés)C# (738 forks)batch import
Help wantedImprovementUp-for-grabs

Métriques du dépôt

Stars
 (3 742 stars)
Métriques de merge PR
 (Merge moyen 1j 1h) (25 PRs mergées en 30 j)

Description

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.)

Guide contributeur

Direction de recherche
Examinez l'implémentation de Path.FullPath dans le code source de Cake et assurez vous qu'elle utilise le séparateur de répertoire natif du système d'exploitation (Path.DirectorySeparatorChar) au lieu d'une barre oblique codée en dur. Vérifiez ensuite que tous les tests réussissent sous Windows.
Stack technique
csharp
Domaine
build system
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70