Azure / Azure/azure-sdk-tools

Apply analyzer to mgmt plane

Open
#6,644 0 comments 0 reactions 0 assignees View on GitHub
.NETClientSDKAnalyzers Client
Dominant language
C#
Stars
135
Forks
260
Avg merge
3d 1h
Merged PRs (30d)
143

Description

We only check DPG client whose name ends with "Client" https://github.com/Azure/azure-sdk-tools/blob/79322c19c69d2cadca592840ac5abf6d2949e829/src/dotnet/Azure.ClientSdk.Analyzers/Azure.ClientSdk.Analyzers/ClientAnalyzerBase.cs#L21. We should remove this condition and check mgmt as well.

Contributor guide

Open the contributing guide

Research direction

Start in src/dotnet/Azure.ClientSdk.Analyzers/Azure.ClientSdk.Analyzers/ClientAnalyzerBase.cs at the linked condition. Inspect how the analyzer identifies clients, then remove the name-suffix restriction so management-plane clients are checked as well. Done means the analyzer applies to both DPG clients and management-plane clients; run the relevant analyzer tests if you locate them.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
tooling
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.