Open phantomtypist opened 7 years ago
@phantomtypist I stumbled across this issue just now because I realized that my personal settings for indentation messed up the formatting of code that was merged from my recent pull requests. My initial thought was "Hey, let's set up an editorconfig file for the project" and was going to create a new issue for it. But then I saw you've already suggested something similar.
It appears that the StyleCop extension is no longer recommended/supported for VS 2015+, replaced by the StyleCopAnalyzers project, so I'd recommend we go that route. My knowledge of StyleCop is very limited, so I will defer to you for your preferred configuration and the actual integration with the project.
Q: Do StyleCop and an .editorconfig file have a place together? It seems like StyleCop is a warning/error generator, whereas we could actually set editor defaults with the config file, and thus both help new contributors from the get-go and prevent deviation from the style rules down the road? Hopefully that makes sense...
Go ahead and add an EditorConfig now. I'll worry about the StyleCop later on.... ultra-low priority.
The Why
Verbiage from the actual StyleCop
I understand that everybody has their own style in how they code, but I think it's beneficial for open source projects like this one to settle on a set of styles and conventions with regards to code. StyleCop rules by default follow most of the standard guidelines set forth for C#. I am not opposed to adding specific rules to conform to something that already exists in this project because I understand it was a port from the Java client and there may be some vestigial styles that came from the Java world.
References: