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

Issue 93050044: code review 93050044: go.talks: remove present tool (Closed)

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

Description

go.talks: remove present tool Present has been moved to code.google.com/p/go.tools/cmd/present. This change leaves behind a trivial package main that prints a message explaining the move.

Patch Set 1 #

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

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

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -2050 lines) Patch
R present/appengine.go View 1 1 chunk +0 lines, -24 lines 0 comments Download
R present/dir.go View 1 1 chunk +0 lines, -281 lines 0 comments Download
R present/doc.go View 1 1 chunk +0 lines, -29 lines 0 comments Download
R present/js/jquery-ui.js View 1 1 chunk +0 lines, -6 lines 0 comments Download
R present/local.go View 1 1 chunk +0 lines, -122 lines 0 comments Download
A present/moved.go View 1 1 chunk +22 lines, -0 lines 0 comments Download
R present/play.go View 1 1 chunk +0 lines, -43 lines 0 comments Download
R present/static/article.css View 1 1 chunk +0 lines, -136 lines 0 comments Download
R present/static/dir.css View 1 1 chunk +0 lines, -186 lines 0 comments Download
R present/static/dir.js View 1 1 chunk +0 lines, -41 lines 0 comments Download
R present/static/favicon.ico View 1 Binary file 0 comments Download
R present/static/print.css View 1 1 chunk +0 lines, -51 lines 0 comments Download
R present/static/slides.js View 1 1 chunk +0 lines, -516 lines 0 comments Download
R present/static/styles.css View 1 1 chunk +0 lines, -453 lines 0 comments Download
R present/templates/action.tmpl View 1 1 chunk +0 lines, -46 lines 0 comments Download
R present/templates/article.tmpl View 1 1 chunk +0 lines, -58 lines 0 comments Download
R present/templates/slides.tmpl View 1 1 chunk +0 lines, -58 lines 0 comments Download

Messages

Total messages: 5
adg
Hello golang-codereviews@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go.talks
9 years, 11 months ago (2014-05-05 21:47:32 UTC) #1
bradfitz
LGTM On May 5, 2014 5:47 PM, <adg@golang.org> wrote: > Reviewers: golang-codereviews, > > Message: ...
9 years, 11 months ago (2014-05-05 21:49:46 UTC) #2
adg
*** Submitted as https://code.google.com/p/go/source/detail?r=519b84408c06&repo=talks *** go.talks: remove present tool Present has been moved to code.google.com/p/go.tools/cmd/present. ...
9 years, 11 months ago (2014-05-05 21:55:20 UTC) #3
kortschak
On 2014/05/05 21:55:20, adg wrote: > *** Submitted as > https://code.google.com/p/go/source/detail?r=519b84408c06&repo=talks *** > > go.talks: ...
9 years, 11 months ago (2014-05-08 10:29:19 UTC) #4
adg
9 years, 11 months ago (2014-05-08 23:22:38 UTC) #5
On 8 May 2014 20:29, <dan.kortschak@adelaide.edu.au> wrote:

> Should one of this pair of CLs have "Fixes issue 7881"?


Yeah, one should have. Close the issue. Thanks for the reminder.
Sign in to reply to this message.

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