Skip to content

[Internal] Code Quality: Adds Exception Analyzers#5082

Merged
kirankumarkolli merged 2 commits intoAzure:masterfrom
msft-tsharp:users/trsharp/enable-code-analyzers
Mar 26, 2025
Merged

[Internal] Code Quality: Adds Exception Analyzers#5082
kirankumarkolli merged 2 commits intoAzure:masterfrom
msft-tsharp:users/trsharp/enable-code-analyzers

Conversation

@msft-tsharp
Copy link
Copy Markdown
Contributor

@msft-tsharp msft-tsharp commented Mar 21, 2025

Description

In order to detect and prevent certain usages of exceptions within the codebase, analyzers have been added to show messages related to potentially incorrect usage.

Added Microsoft.Codex.Analyzers v1.0.5 nuget package
Added .globalconfig file as a baseline configuration

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Closing issues

@msft-tsharp msft-tsharp changed the title [Code Analysis] Enabling Exception Analyzers [Internal] Code Analysis: Add Exception Analyzers Mar 21, 2025
@msft-tsharp msft-tsharp changed the title [Internal] Code Analysis: Add Exception Analyzers [Internal] Analyzer: Add Exception Analyzers Mar 21, 2025
@msft-tsharp msft-tsharp changed the title [Internal] Analyzer: Add Exception Analyzers [Internal] Code Quality: Adds Exception Analyzers Mar 21, 2025
@msft-tsharp msft-tsharp force-pushed the users/trsharp/enable-code-analyzers branch from 3ee146c to 9c635c7 Compare March 21, 2025 21:28
@kirankumarkolli
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@aavasthy
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@kirankumarkolli kirankumarkolli merged commit 0469e16 into Azure:master Mar 26, 2025
23 of 26 checks passed
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.

3 participants