Skip to content

Sync eng/common directory with azure-sdk-tools for PR 12444#6779

Merged
azure-sdk merged 1 commit intomainfrom
sync-eng/common-change-cpex-attestation-pipeline-12444
Oct 10, 2025
Merged

Sync eng/common directory with azure-sdk-tools for PR 12444#6779
azure-sdk merged 1 commit intomainfrom
sync-eng/common-change-cpex-attestation-pipeline-12444

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

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

@azure-sdk azure-sdk requested a review from a team as a code owner October 10, 2025 17:47
@azure-sdk azure-sdk requested review from Copilot and smw-ms October 10, 2025 17:47
@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 Oct 10, 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 12444, removing unused parameters from PowerShell functions to simplify their interfaces.

  • Removed unused parameters from two DevOps work item helper functions
  • Simplified function signatures by removing conditional logic based on unused parameters
  • Cleaned up query building logic that was dependent on the removed parameters

@smw-ms
Copy link
Copy Markdown
Member

smw-ms commented Oct 10, 2025

/check-enforcer override

@azure-sdk azure-sdk merged commit 7f5ab67 into main Oct 10, 2025
44 checks passed
@azure-sdk azure-sdk deleted the sync-eng/common-change-cpex-attestation-pipeline-12444 branch October 10, 2025 18:48
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