search

dennisdoomen / CSharpGuidelines

A set of coding guidelines for C# 9.0, design principles and layout rules for improving the overall quality of your code development.
https://www.csharpcodingguidelines.com
Other
746 stars 271 forks source link

Updated naming to match with Resharper #177

Closed bkoelman closed 6 years ago

bkoelman commented 6 years ago

We have discussed this before. However it turns out there's been a bug in Resharper for many years, which prevents users from changing Resharper's default behavior. And a fix is still not implemented nor planned (feel free to upvote the bug report).

So I updated texts to match with the default Resharper settings for now.

dennisdoomen commented 6 years ago

You really want to update our guideline to match a bug in R#/Rider?

bkoelman commented 6 years ago

No, to match the default setting from Resharper. The bug is only preventing to override that default in obvious ways.

It's not helpful when someone is writing event handler code, then gets a squiggle from CSharpGuidenlinesAnalyzer each time because the naming does not match our pattern. That person then tries to adapt the Resharper setting and gets stuck.

Given this unfortunate combination of things, makes me wonder why we are deviating at all?

dennisdoomen commented 6 years ago

Because it's not English? 🤔

bkoelman commented 6 years ago

That's probably because JetBrains is Russian at heart :)