Update update-dependencies-internal pipelines to accept comma-delimited stage containers#7031
Update update-dependencies-internal pipelines to accept comma-delimited stage containers#7031
Conversation
…ed stage containers Co-authored-by: lbussell <36081148+lbussell@users.noreply.github.com>
Co-authored-by: lbussell <36081148+lbussell@users.noreply.github.com>
| continue | ||
| } | ||
| Write-Host "Processing stage container: $stageContainer" | ||
| dotnet run --project eng/update-dependencies/update-dependencies.csproj -- ` |
There was a problem hiding this comment.
@copilot Instead of running update dependencies in a loop, have the command itself take the list of containers. It should create separate commits for each update, but not separate pull requests.
There was a problem hiding this comment.
Done in 3ec1cca. The from-staging-pipeline command now accepts a comma-delimited list of stage containers. It processes each container sequentially, creates a separate commit for each update, and creates a single pull request containing all commits.
…f stage containers with separate commits per container Co-authored-by: lbussell <36081148+lbussell@users.noreply.github.com>
Co-authored-by: lbussell <36081148+lbussell@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
Co-authored-by: lbussell <36081148+restore` (dns block)
Co-authored-by: lbussell <36081148+restore` (dns block)
|
Understand the codebase
FromStagingPipelineCommand.csandFromStagingPipelineOptions.csUpdate pipelines to accept comma-delimited stage containers instead of pipeline resource
update-dependencies-internal-official.ymlto acceptstageContainersparameterupdate-dependencies-internal-unofficial.ymlto acceptstageContainersparameterupdate-dependencies-internal.ymltemplate to pass the list to the commandUpdate from-staging-pipeline command to handle multiple stage containers
FromStagingPipelineOptions.csto accept comma-delimited list (StageContainersproperty)GetStageContainerList()method to parse the comma-delimited stringFromStagingPipelineCommand.csto:SyncInternalReleaseCommand.csto use new property nameUpdate tests
FromStagingPipelineCommandTests.csfor new APIGetStageContainerList()parsingVerification
GetStageContainerList()consistently)Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.