On Windows, resolving temp directory path incorrectly, doubling the path

[godefroi]

I'm seeing this error in each of my calls to this task:

Run zyborg/dotnet-tests-report@v1.3.0
"C:\Program Files\PowerShell\7\pwsh.exe" -f D:\a\_actions\zyborg\dotnet-tests-report\v1.3.0/action.ps1
New-Item: D:\a\_actions\zyborg\dotnet-tests-report\v1.3.0\action.ps1:40
Line |
  40 |  New-Item -Name $tmpDir -ItemType "directory" -Force
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The filename, directory name, or volume label syntax is incorrect. :
     | 'D:\a\myrepo\myrepo\D:\a\myrepo\myrepo'

[ebekker]

Hmm, interesting, seems to be a problem resolving the full path on Windows, I don't think I've tested this action out on Windows yet, I'll need to make adjustments.

[godefroi]

The action still works, but that error shows up every time it runs.

[ebekker]

Actually, I forgot that I am testing on Windows as well and I didn't run into this issue.

Can you point me to a public example you ran that exhibits this error?

[McManning]

I've been playing around with this action myself and have ran into the same issue.

Here's a public test run from today: https://github.com/McManning/tmp-release-ci/pull/3/checks?check_run_id=2300136665#step:7:11

[ebekker]

I'm not really sure the root cause of this issue, but I've changed the code to try to compute the temp dir using an alternate approach -- if you guys can retry with the latest release and let me know if it fixes it.

[ebekker]

I think the latest point release finally addresses the issue.

[McManning]

I think the latest point release finally addresses the issue.

I've re-ran my tests and can confirm. Thanks!

[ebekker]

Was still not quite right as per #24.