Skip to content
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

chore: fix ReferenceBuilder path handling for VS Debug and dotnet run #90

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

guitarrapc
Copy link
Contributor

@guitarrapc guitarrapc commented Feb 13, 2024

tl;dr;

Current ReferenceBuilder output path is working for VS Debug run, but not for dotnet run --project sandbox\ReferenceBuilder\ReferenceBuilder.csproj. This PR fix to run on both.

$ dotnet run --project sandbox\ReferenceBuilder\ReferenceBuilder.csproj
Unhandled exception. System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\docs\reference_factory.md'.
   at Microsoft.Win32.SafeHandles.SafeFileHandle.CreateFile(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
   at System.IO.File.OpenHandle(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.File.WriteToFile(String path, FileMode mode, String contents, Encoding encoding)
   at Program.<Main>$(String[] args) in C:\github\cysharp\R3\sandbox\ReferenceBuilder\Program.cs:line 6

TODO

  • Run on VS Debug Run
(1175, 1268)
(1300, 1585)
  • Run on dotnet run
$ dotnet run --project sandbox\ReferenceBuilder\ReferenceBuilder.csproj
(1175, 1268)
(1300, 1585)

@guitarrapc guitarrapc merged commit c9da69d into main Feb 13, 2024
@guitarrapc guitarrapc deleted the fix/reference branch February 13, 2024 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant