Skip to content

Conversation

@zhiyuanliang-ms
Copy link
Member

Why this PR?

Add code format check

Visible changes

Linting rules are defined in .editorconfig file.

property is added to the csproj file and then all code format issues will be reported during build process. In the editorconfig file, it sets the severity of IDE0055 and some other formating rules to error.
This will make build fail if the formating rule is not followed.

@juniwang
Copy link
Member

@zhiyuanliang-ms
Copy link
Member Author

zhiyuanliang-ms commented Jul 15, 2024

what's the purpose of this file: https://github.com/Azure/AppConfiguration-DotnetProvider/blob/main/AzureAppConfigurationRules.ruleset ?

@juniwang This file is used for enforce CA2007 check. #410

Code analysis is a big topic. It includes two aspects: code quality and code style.

image

CA2007 belongs to code quality check.

Even for code style, there are 4 aspects as shown in the screenshot below.

image

In this PR, we only focus on the formatting rules.

@juniwang
Copy link
Member

what's the purpose of this file: https://github.com/Azure/AppConfiguration-DotnetProvider/blob/main/AzureAppConfigurationRules.ruleset ?

@juniwang This file is used for enforce CA2007 check. #410

Code analysis is a big topic. It includes two aspects: code quality and code style.

image

CA2007 belongs to code quality check.

Even for code style, there are 4 aspects as shown in the screenshot below.

image

In this PR, we only focus on the formatting rules.

OK,

@juniwang juniwang closed this Jul 17, 2024
@juniwang juniwang reopened this Jul 17, 2024
@zhiyuanliang-ms zhiyuanliang-ms merged commit 08e5930 into main Jul 26, 2024
@zhiyuanliang-ms zhiyuanliang-ms deleted the zhiyuanliang/add-linting-rule branch July 26, 2024 05:01
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.

4 participants