[
https://issues.apache.org/jira/browse/AVRO-3329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17484450#comment-17484450
]
ASF subversion and git services commented on AVRO-3329:
-------------------------------------------------------
Commit 4cd90d08a97d3d9dbc74113efd6df142aa749a3d in avro's branch
refs/heads/branch-1.11 from Kyle Schoonover
[ https://gitbox.apache.org/repos/asf?p=avro.git;h=4cd90d0 ]
AVRO-3329 Added omitted braces to CodeGen class (#1480)
* AVRO-3329 Added omitted braces to CodeGen class
* revert names?.Contains null check
Co-authored-by: Kyle T. Schoonover <[email protected]>
(cherry picked from commit 5eff9a7f503ab0de550c99b6b46ac628ca9d817d)
> Add omitted braces in CodeGen class
> -----------------------------------
>
> Key: AVRO-3329
> URL: https://issues.apache.org/jira/browse/AVRO-3329
> Project: Apache Avro
> Issue Type: Improvement
> Components: csharp
> Affects Versions: 1.11.0
> Reporter: Kyle Schoonover
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 50m
> Remaining Estimate: 0h
>
> Styling improvement. It is standard that when there is an if statement that
> the following line of code is executed. However this can cause confusion
> when maintaining a code base. This would require all statements with
> optional braces to include the braces.
>
> Style Rule Reference: [StyleCopAnalyzers/SA1503.md at master ·
> DotNetAnalyzers/StyleCopAnalyzers
> (github.com)|https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1503.md]
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
