Keyboard Shortcuts

File
u :up to issue
m :publish + mail comments
M :edit review message
j / k :jump to file after / before current file
J / K :jump to next file with a comment after / before current file
Side-by-side diff
i :toggle intra-line diffs
e :expand all comments
c :collapse all comments
s :toggle showing all comments
n / p :next / previous diff chunk or comment
N / P :next / previous comment
<Up> / <Down> :next / previous line
<Enter> :respond to / edit current comment
d :mark current comment as done
Issue
u :up to list of issues
m :publish + mail comments
j / k :jump to patch after / before current patch
o / <Enter> :open current patch in side-by-side view
i :open current patch in unified diff view
Issue List
j / k :jump to issue after / before current issue
o / <Enter> :open current issue
# : close issue
Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :save comment
<Esc> :cancel edit
Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(221)
Issues Repositories Search
Open Issues | Closed Issues | All Issues | Sign in with your Google Account to create issues and add comments

Issue 53670043: code review 53670043: go.blog: update the Thanksgiving doodle post

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 11 months ago by jeff.allen
Modified:
11 years ago
Reviewers:
CC:
adg, golang-codereviews
Visibility:
Public.
go.blog: update the Thanksgiving doodle post Fix the incorrect and missing image. Remove links that do not format correctly. Update the code to match what is running now on appspot. Fixes issue 6908.

Patch Set 1 #

Patch Set 2 : diff -r 1542e3ed4a52 https://code.google.com/p/go.blog #

Patch Set 3 : diff -r 1542e3ed4a52 https://code.google.com/p/go.blog #

Created: 11 years, 11 months ago
Download [raw] [tar.bz2]
Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -55 lines) Patch
M content/from-zero-to-go-launching-on-google.article View 1 3 chunks +54 lines, -55 lines 0 comments Download
A content/from-zero-to-go-launching-on-google_image04.png View 1 Binary file 0 comments Download
Total messages: 5
|
jeff.allen
Hello adg@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.blog
11 years, 11 months ago (2014年01月17日 11:35:41 UTC) #1
Hello adg@golang.org (cc: golang-codereviews@googlegroups.com),
I'd like you to review this change to
https://code.google.com/p/go.blog 
Sign in to reply to this message.
adg
It would be great to break out the code sections into Go source files, so ...
11 years, 11 months ago (2014年01月22日 00:46:46 UTC) #2
It would be great to break out the code sections into Go source files, so they
could be more easily tested (or at least verified as syntactically correct).
This blog post was translated automatically from the HTML, that's why the code
is in-line in the first place.
Are you able to do that?
Sign in to reply to this message.
jeff.allen
Sure, but it will be a few days, maybe more than a week. If you ...
11 years, 11 months ago (2014年01月22日 14:23:18 UTC) #3
Sure, but it will be a few days, maybe more than a week. If you want this
finished now, might be best to take this as it is, and then do the
code-separation yourself.
Sign in to reply to this message.
adg
No problem. Take your time. On 23 January 2014 01:23, <jeff.allen@gmail.com> wrote: > Sure, but ...
11 years, 11 months ago (2014年01月22日 23:13:29 UTC) #4
No problem. Take your time.
On 23 January 2014 01:23, <jeff.allen@gmail.com> wrote:
> Sure, but it will be a few days, maybe more than a week. If you want
> this finished now, might be best to take this as it is, and then do the
> code-separation yourself.
>
> https://codereview.appspot.com/53670043/
>
Sign in to reply to this message.
gobot
R=close To the author of this CL: The Go project has moved to Gerrit Code ...
11 years ago (2014年12月19日 05:18:07 UTC) #5
R=close
To the author of this CL:
The Go project has moved to Gerrit Code Review.
If this CL should be continued, please see the latest version of
https://golang.org/doc/contribute.html for instructions on
how to set up Git and the Go project's Gerrit codereview plugin,
and then create a new change with your current code.
If there has been discussion on this CL, please give a link to it
(golang.org/cl/53670043 is best) in the description in your
new CL.
Thanks very much.
Sign in to reply to this message.
|
Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b

AltStyle によって変換されたページ (->オリジナル) /