Results 9 comments of Dan Roberts

I still don't understand the code well enough to have made much head way but it looks like the ValidateNullability method should pass in the "property" reference when it calls...

One of the problems that I have is that there is nothing tying these bindings to a particular version of React and so the project could be using a version...

I'll look to change those name spaces for consistency. In relation to the dependencies - analysers run within the context of Visual Studio and not the solution that you have...

Sounds good. And I'm totally behind ImmutableCollections not getting into the library which already has its own immutable collections!

@Lonli-Lokli - I _think_ that's correct; that it's possible to have analyser assemblies appear in the build output of the application that has the analyser loaded into one (or more)...

Agreed. Hopefully, if the analyser is included in the main library (like the xunit analysers are) then no Readme will even necessary! We'll just test it out thoroughly and then...

I realised that I didn't add a note here specifically to say that I tested as @Lonli-Lokli suggested and the build output of a project that includes the analyser is...

I wondered if it would be possible to replace _all_ interpolated string sections in such a way that it did not try to change them.. With Cloud Translate, it looks...

That's great! I'll try it tomorrow. Thanks!