I tried very hard to keep the bulk of changes with like changes, and isolate things of a kind to individual commits. Only the final couple of commits are bulk cleanup with Resharper's code cleanup feature, where type members are reorganized and small things I missed in my manual pass are addresses.
Reviewing individual commits will probably reduce the cognitive load substantially.
Some example things done:
Curly braces around all code blocks.
Remove calls to this keyword.
Fix spelling
Rename classes_with_underscores to ClassesInPascalCase
This pull request applies coding standards.
I tried very hard to keep the bulk of changes with like changes, and isolate things of a kind to individual commits. Only the final couple of commits are bulk cleanup with Resharper's code cleanup feature, where type members are reorganized and small things I missed in my manual pass are addresses.
Reviewing individual commits will probably reduce the cognitive load substantially.
Some example things done:
And many more.