[
https://issues.apache.org/jira/browse/AVRO-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17484444#comment-17484444
]
ASF subversion and git services commented on AVRO-3324:
-------------------------------------------------------
Commit ef3fb50465f4f44859a40b665f5ca23870299905 in avro's branch
refs/heads/branch-1.11 from Kyle Schoonover
[ https://gitbox.apache.org/repos/asf?p=avro.git;h=ef3fb50 ]
AVRO-3324 Styling - Added braces for if statements (#1475)
Co-authored-by: Kyle T. Schoonover <[email protected]>
(cherry picked from commit f7188bed2888d72a91c27e3ff0e7da63fe4ecc35)
> Add omitted braces in AvroDecimal
> ---------------------------------
>
> Key: AVRO-3324
> URL: https://issues.apache.org/jira/browse/AVRO-3324
> Project: Apache Avro
> Issue Type: Improvement
> Components: csharp
> Affects Versions: 1.11.0
> Reporter: Kyle Schoonover
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 20m
> 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)
