Feels like an improper "improvement"

The original, simplified:

$"{MinDate.ToString(Format)}";

Resharper offers:

$"{MinDate:MM-dd-yy}";

Which is technically correct as Format is a constant but there's a reason we use constants, it shouldn't be offering to eliminate them!

0
2 comments
Official comment

Hello 

There is the same issue on YouTrack https://youtrack.jetbrains.com/issue/RSRP-465157, looks like it has been already fixed in R# 2017.2 EAP6 https://www.jetbrains.com/resharper/eap/

Thanks! 

Avatar
Permanently deleted user

Agreed--I looked on the forum and didn't find anything and didn't think of looking in bug tracking.  That's definitely the same issue.

0

Please sign in to leave a comment.