Syntax formatting based on dotnet format tool#762
Conversation
There was a problem hiding this comment.
Pull Request Overview
This pull request applies syntax formatting changes based on the dotnet format tool to standardize code formatting across the C# codebase.
- Converts traditional namespace declarations to file-scoped namespaces
- Moves opening braces to the same line for class and method declarations
- Replaces
new List<T>()with collection expressions[] - Replaces
new Dictionary<T>()with collection expressions[]
Reviewed Changes
Copilot reviewed 84 out of 85 changed files in this pull request and generated 3 comments.
Show a summary per file
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
| foreach (var item in dynamicResult) { | ||
| var localValue = item[include.LocalKey].ToString(); | ||
| item[include.Name] = children.ContainsKey(localValue) ? children[localValue] : []; |
There was a problem hiding this comment.
The empty collection expression [] requires explicit type inference. Consider using new List<Dictionary<string, object>>() for clarity, as the compiler may not be able to infer the correct type in all contexts.
| item[include.Name] = children.ContainsKey(localValue) ? children[localValue] : []; | |
| item[include.Name] = children.ContainsKey(localValue) ? children[localValue] : new List<Dictionary<string, object>>(); |
|
|
||
| foreach (var item in dynamicResult) { | ||
| var localValue = item[include.LocalKey].ToString(); | ||
| item[include.Name] = children.ContainsKey(localValue) ? children[localValue] : []; |
There was a problem hiding this comment.
The empty collection expression [] requires explicit type inference. Consider using new List<Dictionary<string, object>>() for clarity, as the compiler may not be able to infer the correct type in all contexts.
| item[include.Name] = children.ContainsKey(localValue) ? children[localValue] : []; | |
| item[include.Name] = children.ContainsKey(localValue) ? children[localValue] : new List<Dictionary<string, object>>(); |
| if (alias != null) { | ||
| query.As(alias); | ||
| } | ||
| ; |
There was a problem hiding this comment.
The semicolon after the closing brace is unnecessary and should be removed.
No behavioral changes