Skip to content

[VMR] Remove 'Copy Test NuGet Config' step#19450

Merged
akoeplinger merged 2 commits intomainfrom
akoeplinger-patch-1
Apr 12, 2024
Merged

[VMR] Remove 'Copy Test NuGet Config' step#19450
akoeplinger merged 2 commits intomainfrom
akoeplinger-patch-1

Conversation

@akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented Apr 12, 2024

This step is no longer needed after #19366.

After #19366 the `runTests` variable is true on Windows too so the step to copy the nuget.config for source-build smoke tests started running there, resulting in an error:
> 'cp' is not recognized as an internal or external command, operable program or batch file.

Skip that step when we're not in doing a source-build.
@akoeplinger akoeplinger requested a review from ViktorHofer April 12, 2024 14:05
@akoeplinger akoeplinger changed the title [VMR] Only copy test nuget.config for source-build [VMR] Remove 'Copy Test NuGet Config' step Apr 12, 2024
@akoeplinger akoeplinger enabled auto-merge (squash) April 12, 2024 15:58
@akoeplinger akoeplinger merged commit ef2df86 into main Apr 12, 2024
@akoeplinger akoeplinger deleted the akoeplinger-patch-1 branch April 12, 2024 17:21
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.

2 participants