mhutch / MonoDevelop.MSBuildEditor

Improved MSBuild editing support
Other
215 stars 26 forks source link

The value is not a valid GUID format #174

Closed Suchiman closed 5 months ago

Suchiman commented 5 months ago

The extension erroneously warns that image this is due to this code requiring the B format for some reason https://github.com/mhutch/MonoDevelop.MSBuildEditor/blob/1079e36b274da6d8f97caaa1ddb7aeb42b390882/MonoDevelop.MSBuild/Language/MSBuildDocumentValidator.cs#L636-L637 which only permits guids using hyphens AND curly braces image Unless there's some reason that i'm unaware of, i would suggest just switching to Guid.TryParse to recognize any valid guid.

mhutch commented 5 months ago

I believe it did that because some of the properties typed as guid did require that format, so I'll need to check the other uses of that type. From an MSBuild perspective they're just strings and it's up to the tasks to parse them, so it's difficult to be truly consistent. I'll change the guid type to allow any format supported by TryParse, but we may need another type to represent the constrained GUID format for some other properties.

mhutch commented 5 months ago

Per #181 it turns out this doesn't actually have to be a GUID at all, so I updated the schema to mark it as a string.

However, I went ahead and implemented the analyzer hints, so guid will now allow anything supported by Guid.TryParse(), and I updated various other GUIDs in the schema to use "analyzerHints": { "GuidFormat": "B" }