Roslyn analyzers (extended code analysis) and CSharpier (automatic code formatting) were added to most projects in the Origam C# solution in the Directory.Build.props config file. Some warnings are ignored because they would require too extensive changes in the code that were out of scope of the original task.
All ingnored warning should be reviewed and the useful ones should be enabled. That includes for example:
RCS1203
Use lambda expression instead of anonymous method to modernize code style.
RCS1194
Implement all standard exception constructors for custom exception types.
RCS1059
Avoid empty catch clauses that catch System.Exception to prevent swallowing errors.
RCS1215
Detects expressions that always evaluate to true or false to catch redundant or buggy code.
RCS1155
Use StringComparison when comparing strings to avoid culture and locale issues.
RCS1172
Use read-only auto-implemented properties for values assigned only in constructors.
RCS1044
Remove redundant empty lines for consistent code formatting.
CA2200
Use throw; instead of throw ex; to preserve the original stack trace when rethrowing exceptions.