LGTM On Tuesday, 26 August 2014, <r@golang.org> wrote: > Reviewers: adg, > > Message: > ...
9 years, 8 months ago
(2014-08-26 00:32:11 UTC)
#2
LGTM
On Tuesday, 26 August 2014, <r@golang.org> wrote:
> Reviewers: adg,
>
> Message:
> Hello adg (cc: golang-codereviews@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/go.blog
>
>
> Description:
> go.blog: s/./,/ in constants blog post
>
> Please review this at https://codereview.appspot.com/129630043/
>
> Affected files (+1, -1 lines):
> M content/constants.article
>
>
> Index: content/constants.article
> ===================================================================
> --- a/content/constants.article
> +++ b/content/constants.article
> @@ -332,7 +332,7 @@
>
> .play -edit constants/int2.go /var/
>
> -Similarly, `uint8`, also known as `byte`. has range 0 through 255, so a
> large or negative constant cannot be assigned to a `uint8`:
> +Similarly, `uint8`, also known as `byte`, has range 0 through 255, so a
> large or negative constant cannot be assigned to a `uint8`:
>
> .play -edit constants/int3.go /var/
>
>
>
>
*** Submitted as https://code.google.com/p/go/source/detail?r=ce859502eb88&repo=blog *** go.blog: s/./,/ in constants blog post LGTM=adg R=adg CC=golang-codereviews https://codereview.appspot.com/129630043
9 years, 8 months ago
(2014-08-26 00:32:53 UTC)
#3
Issue 129630043: code review 129630043: go.blog: s/./,/ in constants blog post
(Closed)
Created 9 years, 8 months ago by r
Modified 9 years, 8 months ago
Reviewers:
Base URL:
Comments: 0