Closed DoctorKrolic closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.47%. Comparing base (
b07c100
) to head (aba7cf1
). Report is 2 commits behind head on main.:exclamation: Current head aba7cf1 differs from pull request most recent head 37a5a32. Consider uploading reports for the commit 37a5a32 to get more accurate results
Seems like CI is stuck
Fixes: https://github.com/dotnet/roslyn-analyzers/issues/4539
It turned out that despite
DiagnosticDescriptor
class is fully equatable, it isn't safe to compare instances in a codefix due to localized strings. For whatever reason all localizable strings indiagnostic.Descriptor
are fixed strings while singleton descriptor instances in tha analyzer contain localizable resource strings, so the comparison could never succeed and thus fixes were not shown. I guess, this might be due to serialization-deserialization between devenv and roslyn OOP. To avoid that I made it so that diagnostics are now identified viaRule
property, which is correctly passed between analyzer and a codefix provider. I've made a local setup where I verified, that this actually resolves the problem and the fix is finally shown. I don't really like the proposed fix, but that is a story for another PR)