Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1409)

Issue 66790048: code review 66790048: go.blog: add FOSDEM 2014 videos article (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 2 months ago by adg
Modified:
10 years, 2 months ago
Reviewers:
r, bradfitz
CC:
bradfitz, r, golang-codereviews
Visibility:
Public.

Description

go.blog: add FOSDEM 2014 videos article

Patch Set 1 #

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

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

Total comments: 4

Patch Set 4 : diff -r d18ac1d07ab1 https://code.google.com/p/go.blog #

Unified diffs Side-by-side diffs Delta from patch set Stats (+73 lines, -0 lines) Patch
A content/fosdem14.article View 1 2 3 1 chunk +73 lines, -0 lines 0 comments Download

Messages

Total messages: 9
adg
Hello bradfitz, I'd like you to review this change to https://code.google.com/p/go.blog
10 years, 2 months ago (2014-02-24 00:45:17 UTC) #1
adg
I only included what I judge to be the 3 mont popular videos of the ...
10 years, 2 months ago (2014-02-24 00:46:36 UTC) #2
bradfitz
At least link "a series of talks" to http://video.fosdem.org/2014/K4601/Sunday/ ? On Sun, Feb 23, 2014 ...
10 years, 2 months ago (2014-02-24 00:47:39 UTC) #3
bradfitz
LGTM Oh, the YouTube playlist has them all. On Sun, Feb 23, 2014 at 4:47 ...
10 years, 2 months ago (2014-02-24 00:48:08 UTC) #4
bradfitz
... but you might still want to link to the original webm files as an ...
10 years, 2 months ago (2014-02-24 00:48:41 UTC) #5
adg
PTAL
10 years, 2 months ago (2014-02-24 00:54:59 UTC) #6
bradfitz
LGTM
10 years, 2 months ago (2014-02-24 05:13:15 UTC) #7
r
LGTM https://codereview.appspot.com/66790048/diff/40001/content/fosdem14.article File content/fosdem14.article (right): https://codereview.appspot.com/66790048/diff/40001/content/fosdem14.article#newcode9 content/fosdem14.article:9: At FOSDEM on the 2nd of February 2014 ...
10 years, 2 months ago (2014-02-24 23:14:21 UTC) #8
adg
10 years, 2 months ago (2014-02-25 00:47:40 UTC) #9
*** Submitted as
https://code.google.com/p/go/source/detail?r=533486efe614&repo=blog ***

go.blog: add FOSDEM 2014 videos article

LGTM=bradfitz, r
R=bradfitz, r
CC=golang-codereviews
https://codereview.appspot.com/66790048

https://codereview.appspot.com/66790048/diff/40001/content/fosdem14.article
File content/fosdem14.article (right):

https://codereview.appspot.com/66790048/diff/40001/content/fosdem14.article#n...
content/fosdem14.article:9: At FOSDEM on the 2nd of February 2014 members of the
Go community presented a
On 2014/02/24 23:14:21, r wrote:
> link to FOSDEM

Done.

https://codereview.appspot.com/66790048/diff/40001/content/fosdem14.article#n...
content/fosdem14.article:17:
[[http://www.youtube.com/playlist?list=PLtLJO5JKE5YDKG4WcaNts3IVZqhDmmuBH][as a
YouTube playlist]]. (You can also download them directly from the
On 2014/02/24 23:14:21, r wrote:
> don't change it because people understand this but....
> 
> You can also get them directly at
> 
> is much shorter and just as clear

Done.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b