Skip to content

Conversation

kevinzz6
Copy link
Member

Packages impacted by this PR

Issues associated with this PR

Describe the problem that is addressed by this PR

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

Are there test cases added in this PR? (If not, why?)

Provide a list of related PRs (if any)

Command used to generate this PR:**(Applicable only to SDK release request PRs)

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)

@Copilot Copilot AI review requested due to automatic review settings September 28, 2025 07:18
@kevinzz6 kevinzz6 requested review from qiaozha, MaryGao and a team as code owners September 28, 2025 07:18
Copy link
Contributor

@Copilot 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 updates the code ownership for the Synapse SDK by adding a new maintainer to the ownership configuration.

Key Changes

  • Adds @wanyang7 as an additional code owner for the Synapse SDK alongside the existing owner @joheredi
  • Updates both the Azure SDK owners and service owners sections for consistency

# AzureSdkOwners: @joheredi
/sdk/synapse/ @joheredi
# AzureSdkOwners: @joheredi @wanyang7
/sdk/synapse/ @joheredi @wanyang7
Copy link
Member

@MaryGao MaryGao Sep 29, 2025

Choose a reason for hiding this comment

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

@joheredi @xirzec for data-plane SDKs, will we add service teams as their folder root owners? My understanding is this means service team could prepare release fully by themselves, is this our recommandation?

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