Skip to content

Conversation

@guimafelipe
Copy link
Contributor

If I don't have the nuget.exe directory created, running DevCheck.ps1 -NugetExeUpdate fails as the curl command cannot create the file in the download process (by default, if I don't have the .user directory).

This change makes the script create the directory in case of a first download of the nuget in the computer.

A microsoft employee must use /azp run to validate using the pipelines below.

WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.

For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

@guimafelipe guimafelipe marked this pull request as ready for review November 25, 2025 21:24
@guimafelipe guimafelipe requested a review from Copilot November 25, 2025 21:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes an issue where the DevCheck script fails when attempting to download nuget.exe if the target directory doesn't exist. The script now creates the necessary directory structure before downloading the file.

  • Adds directory existence check before downloading nuget.exe
  • Creates the directory if it doesn't exist to prevent curl download failures

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@guimafelipe guimafelipe requested a review from Copilot November 25, 2025 21:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants