Skip to content
This repository was archived by the owner on Aug 25, 2025. It is now read-only.

document wire.Struct first argument#197

Merged
shantuo merged 2 commits into
google:masterfrom
shantuo:docstruct196
Jun 18, 2019
Merged

document wire.Struct first argument#197
shantuo merged 2 commits into
google:masterfrom
shantuo:docstruct196

Conversation

@shantuo
Copy link
Copy Markdown
Contributor

@shantuo shantuo commented Jun 17, 2019

Fixes #196

@shantuo shantuo requested a review from zombiezen June 17, 2019 17:45
@go-cloud-bot
Copy link
Copy Markdown

go-cloud-bot Bot commented Jun 17, 2019

Please edit the title of this pull request with the name of the affected component, or "all", followed by a colon, followed by a short summary of the change.

@googlebot googlebot added the cla: yes Google CLA has been signed! label Jun 17, 2019
Copy link
Copy Markdown
Contributor

@zombiezen zombiezen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after suggestions

Comment thread wire.go Outdated
Comment thread wire.go Outdated
Comment thread wire.go Outdated
@shantuo shantuo merged commit 89cb901 into google:master Jun 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes Google CLA has been signed!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document wire.Struct's first argument form

3 participants