"After a decade of working with Umbraco, I'm excited to say I'm close to making my first PR! My goal is simply to clean up some Build/StyleCop warnings and modernize bits of code by implementing some of the newer C# features like the null-coalescing assignment operator and ArgumentOutOfRangeExceptions, etc. I'm not overhauling any major functionality—just making incremental improvements. I started by focusing on individual files and planned on making separate commits for each. My first file resulted in 238 changes.
Given this approach, I have a couple of questions:
Is this level of file by file granularity necessary/encouraged/discouraged for each commit?
OR
If I'm tackling a specific warning, like 'IDE0041: Null check can be simplified,' should I apply the fix across the entire solution and then make that a single commit?
I appreciate your insights!"