James Wyatt Cready-Pyle

Results 79 comments of James Wyatt Cready-Pyle

See https://github.com/timostamm/protobuf-ts/discussions/324

Possibly provide a new package for parsing the [currently unspecified](https://github.com/protocolbuffers/protobuf/issues/8636) text protobuf format.

Regarding the `FieldMask` stuff on the `IMessageType`: wouldn't that require that the runtime (I assume) had access to a pre-generated generated `FieldMask` `MessageType`? How would that work?

While investigating the FieldMask stuff I noticed that the existing implementations (namely the [Java](https://developers.google.com/protocol-buffers/docs/reference/java/com/google/protobuf/util/FieldMaskUtil.MergeOptions.html) and [Python](https://googleapis.dev/python/protobuf/latest/google/protobuf/field_mask_pb2.html#google.protobuf.field_mask_pb2.FieldMask.MergeMessage) versions) both have "merge options" wherein the caller is able to configure the merge...

No worries! Thank you again for this wonderful project! It's head and shoulders above anything else currently available.

All the currently supported CLI options for the `protoc` plugin are simply flags (booleans where their presence means `true` and absence means `false`). I _believe_ it should be possible to...

The issue appears to be that the [`coverLogicalExpression` visitor](https://github.com/istanbuljs/istanbuljs/blob/aaa6dc506df404083be49ca442e9788156708c93/packages/istanbul-lib-instrument/src/visitor.js#L498) only considers each leaf to be a _single_ branch instead of 2 (one if the expression evaluates to `true` and one...

Wait a second, it looks like this just recently landed behind an option: https://github.com/istanbuljs/istanbuljs/pull/629

The flat structure would end up making typescript's job much harder (and more expensive to type check). Imagine a proto message with 10 oneofs each with 10 fields. With the...