Skip to content

Conversation

@qwordy
Copy link
Member

@qwordy qwordy commented Mar 24, 2021

Description

Check format of service_name.json. Command and AzureServiceName are required. Others are optional.
Each highest level command group should have reference in service_name.json.

https://ceapex.visualstudio.com/Engineering/_workitems/edit/327925
https://docs.microsoft.com/en-us/cli/azure/azure-cli-reference-for-network?view=azure-cli-latest

[Core] Linter rule for service_name.json

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@yungezz yungezz requested a review from kairu-ms March 25, 2021 02:00
@yungezz
Copy link
Member

yungezz commented Mar 25, 2021

hi @qwordy, as offline synced, pls share with the team context of the rule, what's it for before merging it. thanks

@qwordy
Copy link
Member Author

qwordy commented Mar 25, 2021

hi @qwordy, as offline synced, pls share with the team context of the rule, what's it for before merging it. thanks

OK

@qwordy
Copy link
Member Author

qwordy commented Mar 25, 2021

Test Extensions Loading Python36 keeps failing.

Copy link
Contributor

@kairu-ms kairu-ms left a comment

Choose a reason for hiding this comment

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

It's clear now.

@qwordy qwordy merged commit 616ff36 into Azure:dev Apr 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants