Skip to content

Conversation

@kazrael2119
Copy link
Member

@kazrael2119 kazrael2119 commented Jul 24, 2024

@kazrael2119 kazrael2119 requested review from a team, MaryGao and qiaozha as code owners July 24, 2024 02:18
@github-actions github-actions bot added the Mgmt This issue is related to a management-plane library. label Jul 24, 2024
@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

@azure/arm-containerservice

@qiaozha
Copy link
Member

qiaozha commented Jul 25, 2024

image Not sure why this pr is still requesting review from azure-sdk-for-js-core? Feel like it's because of the api.md file change, but since I am on the owner list, curious why it's not working ? @jeremymeng @xirzec

@qiaozha qiaozha merged commit fe18ae3 into Azure:main Jul 25, 2024
@jeremymeng
Copy link
Member

@qiaozha it is the tsconfig.json. I will try adding another entry

@jeremymeng
Copy link
Member

@qiaozha I guess the pattern didn't work

/sdk/**/arm-*/tsconfig.json @qiaozha @MaryGao @Azure/azure-sdk-for-js-core

@jeremymeng
Copy link
Member

#30527

@kazrael2119 kazrael2119 deleted the containerservice-stable branch July 26, 2024 01:55
@qiaozha
Copy link
Member

qiaozha commented Jul 26, 2024

@jeremymeng thanks for taking care of it.

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

Labels

Mgmt This issue is related to a management-plane library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants