David Castro comments

Results 102 comments of


                                            David Castro

Add protoc-gen-go-setters to Other Utilities third_party.md

You'll need to sign the CLA before we can accept a PR.

Message FieldNameEntry not allowed as map value type for field_name

I assume we want a documentation bandaid in the meantime? I can add something to the proto and proto3 topics.

Message FieldNameEntry not allowed as map value type for field_name

I've sent a documentation update out for review. I don't think that it will fully resolve the issue, but at least the behavior will be documented.

Fix protobuf examples at editions/overview.md

Thanks for sending this PR to fix the syntax problems. I'll work on getting this incorporated.

Fix protobuf examples at editions/overview.md

This fix was incorporated in [this PR that bundled various updates](https://github.com/protocolbuffers/protocolbuffers.github.io/pull/161). Thanks for sending the update!

Update gotutorial.md

Thanks for your contributiont to improving/fixing the docs! This PR is missing a CLA. Once you've agreed to the CLA, I'll be able to assess the contribution.

Update gotutorial.md

This PR is missing a CLA. Once you've agreed to the CLA, I'll be able to assess the contribution.

Update gotutorial.md

This issue is still indicating that the **cla/google** check is failing. I cannot merge the PR without the CLA test passing.

Update gotutorial.md

That worked; thanks! I'll see if I can get this done this week.

Update gotutorial.md

This fix was incorporated in [this PR that bundled various updates](https://github.com/protocolbuffers/protocolbuffers.github.io/pull/161). Thanks for sending the update!