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

Issue 6218069: code review 6218069: cmd/api: add api/next.txt (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 12 months ago by bradfitz
Modified:
11 years, 12 months ago
Reviewers:
CC:
golang-dev, r
Visibility:
Public.

Description

cmd/api: add api/next.txt This quiets all.bash noise for upcoming features we know about. The all.bash warnings will now only print for things not in next.txt (or in next.txt but not in the API). Once an API is frozen, we rename next.txt to a new frozen file (like go1.txt) Fixes issue 3651

Patch Set 1 #

Patch Set 2 : diff -r fa807f847959 https://go.googlecode.com/hg #

Patch Set 3 : diff -r fa807f847959 https://go.googlecode.com/hg #

Total comments: 2

Patch Set 4 : diff -r 9761320d1318 https://go.googlecode.com/hg #

Unified diffs Side-by-side diffs Delta from patch set Stats (+515 lines, -35 lines) Patch
A api/README View 1 2 3 1 chunk +11 lines, -0 lines 0 comments Download
A api/next.txt View 1 2 3 1 chunk +428 lines, -0 lines 0 comments Download
M src/cmd/api/goapi.go View 1 2 3 2 chunks +74 lines, -33 lines 0 comments Download
M src/run.bash View 1 1 chunk +1 line, -1 line 0 comments Download
M src/run.bat View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
bradfitz
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg
11 years, 12 months ago (2012-05-22 23:42:53 UTC) #1
r
LGTM http://codereview.appspot.com/6218069/diff/5001/api/README File api/README (right): http://codereview.appspot.com/6218069/diff/5001/api/README#newcode9 api/README:9: features which may become the next version. It ...
11 years, 12 months ago (2012-05-23 00:00:34 UTC) #2
bradfitz
11 years, 12 months ago (2012-05-23 01:41:26 UTC) #3
*** Submitted as http://code.google.com/p/go/source/detail?r=1098c239b9ad ***

cmd/api: add api/next.txt

This quiets all.bash noise for upcoming features we know about.

The all.bash warnings will now only print for things not in next.txt
(or in next.txt but not in the API).

Once an API is frozen, we rename next.txt to a new frozen file
(like go1.txt)

Fixes issue 3651

R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/6218069
Sign in to reply to this message.

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