Hello adg@golang.org, campoy@google.com (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.talks
9 years, 8 months ago
(2014-07-22 23:40:28 UTC)
#1
https://codereview.appspot.com/113430043/diff/2/2014/hammers.slide File 2014/hammers.slide (right): https://codereview.appspot.com/113430043/diff/2/2014/hammers.slide#newcode66 2014/hammers.slide:66: * go test -cover show the coverage output? https://codereview.appspot.com/113430043/diff/2/2014/hammers.slide#newcode110 ...
9 years, 8 months ago
(2014-07-23 03:40:54 UTC)
#3
no images in the file list :( On Wed, Jul 23, 2014 at 3:40 AM, ...
9 years, 8 months ago
(2014-07-24 07:36:25 UTC)
#6
no images in the file list :(
On Wed, Jul 23, 2014 at 3:40 AM, <josharian@gmail.com> wrote:
> Reviewers: adg, campoy,
>
> Message:
> Hello adg@golang.org, campoy@google.com (cc:
> golang-codereviews@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/go.talks
>
>
> Description:
> go.talks: add OSCON "Gophers with Hammers" talk
>
> Please review this at https://codereview.appspot.com/113430043/
>
> Affected files (+617, -0 lines):
> A 2014/hammers.slide
> A 2014/hammers/codegen.go
> A 2014/hammers/extractiface.go
> A 2014/hammers/extractpath.go
> A 2014/hammers/findthecode.go
> A 2014/hammers/findtheifacedecl.go
> A 2014/hammers/format.go
> A 2014/hammers/fulltype.go
> A 2014/hammers/importpath.go
> A 2014/hammers/types.go
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
Issue 113430043: code review 113430043: go.talks: add OSCON "Gophers with Hammers" talk
(Closed)
Created 9 years, 8 months ago by josharian
Modified 9 years, 8 months ago
Reviewers:
Base URL:
Comments: 8