Fix SA1615/SA1623 warnings

[iamdmitrij]

Pull Request

The issue or feature being addressed

#1290

Details on the issue fix or feature implementation

• Fix SA1615 by adding documentation to return values
• Fix SA1623 by prefixing property docs to Gets/Sets

Confirm the following

• I started this PR by branching from the head of the default branch
• I have targeted the PR to merge into the default branch
• I have included unit tests for the issue/feature
• I have successfully run a local build

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.72%. Comparing base (a31bdc2) to head (bd0bac6).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2091   +/-   ##
=======================================
  Coverage   83.72%   83.72%           
=======================================
  Files         312      312           
  Lines        7126     7126           
  Branches     1054     1054           
=======================================
  Hits         5966     5966           
  Misses        789      789           
  Partials      371      371           

Flag	Coverage Δ
linux	83.72% <ø> (ø)
macos	83.72% <ø> (ø)
windows	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[iamdmitrij]

Noticed that SA1623 can be fixed without suppressing rule. Will push new changes.

[iamdmitrij]

SA1623 is now fixed without .editorconfig suppression.

[martincostello]

Thanks again!