This repository was archived by the owner on Dec 13, 2018. It is now read-only.
This repository was archived by the owner on Dec 13, 2018. It is now read-only.
Clean up property behavior and doc comments for AzureAppServicesDiagnosticsSettings #506
Closed
Description
Some random stuff I noticed:
- The property setters should throw ArgOutOfRange w/ appropriate message for invalid values
BackgroundQueueSize
has no proper doc comment- The non-doc comment comment for
BackgroundQueueSize
has a typo:Gets of sets ...
-->Gets or sets ...
- Would probably be nice for the doc comments to indicate the actual default values, e.g.
The default value for this property is <c>FOO</c>.
or whatever the right doc comment syntax is.
@muratg we should do this for 1.1.0.