New rule: Guid data types should be checked with IsNullGuid()

[tinestaric]

I tiny rule idea I noticed catching recently. If you try to check for an empty guid with
if MyGuid = '' then ..
then you hit a runtime error that guid is not in the correct format (because the '' is not a valid guid).

What if check for this comparison and throw a warning that IsNullGuid should be used?

[Arthurvdv]

Thanks for the suggestion 🙋‍♂️

I've created an new rule for this: LC0087 - Use IsNullGuid() to check for empty Guid values., which is now in the Pre-release version v0.32.1 of the LinterCop.

The rule will also will raise a diagnostic on other operators, like for example MyGuid <> ''.

[Arthurvdv]

The rule is now available in the release version from version v0.33.0 of the LinterCop.