I this "Replace with" suffix is not needed. What about AnotherCase?..?

How is C# doing this? The text is from earlier commit. But we can change it.

ready for review

Hmmm. Not sure I like surfacing the possibility of something completely unrelated, like GetHashCode when we're specifically trying to fix a Record label or DU case. That's likely going to result in quite a few people logging bugs with the feedback tool.

The GetHashCode can probably be filtered out here since it's not a static method. But this is unrelated to this PR here since without this PR it would only propose the GetHashCode (which is actually worse)

Okay. I think if we can filter out non-static members on types then all the suggestions will seem relevant.

just took a quick look. this is not easy to determine at this location since we don't know if we are invoked in static or member position. So since this PR only improves the current situation it should still be merged. I have an idea for the filtering and will apply that later.

Sounds good.

Ok all tests fixed

@cartermp since I saw your list on twitter. This pr is not for the analyzer. It's actually just the error message improvement.
But since the code fix stuff is already merged this error message is automatically used in the analyzer. Ionide will profit as well automatically when this got merged to FCS.

Good to know!