Path normalization should not check drive letter on non-Windows
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 76/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- csharp
- Domain
- operating-systems
Research direction
Start in TruePath/PathStrings.cs at lines 176-191 and inspect how path normalization checks drive letters. Reproduce the behavior for C:/ and /: on Unix, then make the check OS-specific or independently controllable; done means these paths no longer break on Unix while drive-letter handling remains correct where applicable.
Written by the indexing model from the issue text.
Description
It seems this will break for paths like C:/ or /: on Unix.
Guard this per OS. Best of all, make the OS or bool checkDriveLetter a parameter, to keep the function behavior independent of OS.
- Dominant language
- C#
- Stars
- 104
- Forks
- 19
- Avg merge
- 13h 42m
- Merged PRs (30d)
- 9
Getting set up
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ForNeVeR/TruePath
-
enhancement good first issue help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
ForNeVeR/TruePath#165 · 2 comments ·
Maintainers usually reply within 1 day
-
bug help wanted
Difficulty 3/5 1-2 days Newbie friendliness 68/100
ForNeVeR/TruePath#226 · 1 comment ·
Maintainers usually reply within 1 day
-
LocalPath: prefix matching needs a four-way path kind, not IsPathRootedPossibly taken @ForNeVeR claimed this 5 days ago. Openbug
Difficulty 4/5 3-5 days Newbie friendliness 55/100
ForNeVeR/TruePath#224 · 1 assignee ·
Maintainers usually reply within 1 day
-
enhancement good first issue help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
Maintainers usually reply within 1 day
-
Dependency DashboardOpendashboard
Difficulty 3/5 1-2 days Newbie friendliness 15/100
Maintainers usually reply within 1 day
All issues in ForNeVeR/TruePath
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
fluentassertions/fluentassertions#3353 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
Difficulty 2/5 Half a day Newbie friendliness 78/100
unoplatform/uno#24769 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
AvaloniaUI/Avalonia#22323 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
microsoft/onnxruntime-genai#2633 ·
Maintainers usually reply within 1 day