[CSHARP-4081] Review .editorconfig Created: 01/Mar/22  Updated: 31/Mar/22

Status: Backlog
Project: C# Driver
Component/s: Build
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Unknown
Reporter: Robert Stam Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

We are currently getting a lot of mostly bogus warnings related to .editorconfig.

Most of these seem to be mismatches between .editorconfig and our coding style.

.editorconfig needs to be modified to reduce the number of warnings.

In some cases it might be warranted to make formatting changes to the code instead, but that should be the exception.



 Comments   
Comment by Robert Stam [ 01/Mar/22 ]

Examples of things where we need to update .editorconfig and not our code:

Message	IDE0032	Use auto property
Message	IDE0008	Use explicit type instead of 'var'
Message	IDE1006	Naming rule violation: Missing prefix: 's_'

Examples of things that we would like to do going forward but where it's unclear whether we want to make extensive changes to existing code:

Message	IDE0034	'default' expression can be simplified
Message	IDE0016	Null check can be simplified

Generated at Wed Feb 07 21:47:10 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.