Jon Skeet

Results 749 comments of Jon Skeet

That's fascinating in terms of the samples. Those wouldn't be representable in .vkit files, as I don't have access to samples (either loading or saving) as far as I'm aware....

Right. I need to have a think about whether it would make sense to try to include this within the vkit format, or have a whole separate format for td0....

Hacky fix which works for me, basically the manual version of #8135 until there's a release containing that: - Download System.Memory NuGet package version 4.5.5 - Extract `lib/net461/System.Memory.dll` into the...

@CalvinWilkinson: I'm afraid I haven't tried using docfx that way at all.

@CalvinWilkinson: I just unzip the release and execute `docfx.exe` directly (always on a Windows box, which may make a difference).

Four years after being reported, this is still an issue :(

Point 2 is a failure on my part - I'll have a look at the schema validation code tomorrow. (Any NullReferenceException is a code bug...) Thanks for flagging it!

https://github.com/googleapis/google-cloudevents-dotnet/pull/98 improves the .NET tooling - the validation would fail with this instead: ```text Google.Protobuf.InvalidProtocolBufferException: Unknown field: data at Google.Protobuf.JsonParser.Merge(IMessage message, JsonTokenizer tokenizer) at Google.Protobuf.JsonParser.Merge(IMessage message, TextReader jsonReader) at Google.Protobuf.JsonParser.Parse(TextReader...

Okay, I've now performed the test. Fields added with `build_repo_error`, `build_storage_error` and `storage_error` (in the three places mentioned above). Here's the change in schemas: ##In BuildEventData.json: In `definitions/RepoSource/properties` there's a...

@grant: I would suggest that we should ask each language owner to consider the developer experience and decide whether it's sufficient or not. The example I gave in Java (with...