Skip to content

Sync eng/common directory with azure-sdk-tools for PR 11974#25187

Merged
praveenkuttappan merged 5 commits intomainfrom
sync-eng/common-add_optional_release_plan_id-11974
Sep 6, 2025
Merged

Sync eng/common directory with azure-sdk-tools for PR 11974#25187
praveenkuttappan merged 5 commits intomainfrom
sync-eng/common-add_optional_release_plan_id-11974

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#11974 See eng/common workflow

@azure-sdk azure-sdk requested review from Copilot and praveenkuttappan and removed request for praveenkuttappan September 5, 2025 01:00
@azure-sdk azure-sdk requested a review from a team as a code owner September 5, 2025 01:00
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Sep 5, 2025
Copy link
Copy Markdown
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 syncs the eng/common directory with azure-sdk-tools for PR 11974, adding new PowerShell functionality for updating pull request information in Azure DevOps release plans.

  • Adds a new PowerShell script to update pull request URLs and statuses in release plan work items
  • Introduces helper functions for managing release plan data and pull request information
  • Implements GitHub comment integration to provide release plan context on pull requests

Reviewed Changes

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

File Description
eng/common/scripts/Update-PullRequest-In-ReleasePlan.ps1 New script that updates pull request information in release plans and adds contextual comments to GitHub PRs
eng/common/scripts/Helpers/DevOps-WorkItem-Helpers.ps1 Adds helper functions for updating pull request data in release plans and retrieving release plan information

Write-Host "Updated Pull Request URL [$pullRequestUrl] for [$languageName] in Release Plan [$releasePlanWorkItemId]"
}

function Get-ReleasePlan-Link($releasePlanWorkItemId)
Copy link

Copilot AI Sep 5, 2025

Choose a reason for hiding this comment

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

Function name 'Get-ReleasePlan-Link' uses inconsistent naming convention with a hyphen in the middle. PowerShell function names should use approved verbs followed by a noun, such as 'Get-ReleasePlanLink' to maintain consistency with other functions in the codebase.

Copilot uses AI. Check for mistakes.
@azure-sdk azure-sdk force-pushed the sync-eng/common-add_optional_release_plan_id-11974 branch 2 times, most recently from dc83451 to 2e46fc2 Compare September 5, 2025 16:50
@azure-sdk azure-sdk force-pushed the sync-eng/common-add_optional_release_plan_id-11974 branch from 2e46fc2 to 31f85b3 Compare September 5, 2025 16:54
@praveenkuttappan praveenkuttappan merged commit 2c0f5d0 into main Sep 6, 2025
11 checks passed
@praveenkuttappan praveenkuttappan deleted the sync-eng/common-add_optional_release_plan_id-11974 branch September 6, 2025 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants