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

Issue 61690044: code review 61690044: ogle: fix A+C so we don't need to maintain contributors (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 2 months ago by r
Modified:
10 years, 2 months ago
Reviewers:
rsc
CC:
rsc, nigeltao
Visibility:
Public.

Description

ogle: fix A+C so we don't need to maintain contributors Put coderefview.cfg in the root and copy it, plus A+C from go.tools. Drop the defaultcc: line from codereview.cfg, at least for now.

Patch Set 1 #

Patch Set 2 : diff -r 491eb7565e88 https://code.google.com/p/ogle #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -19 lines) Patch
M AUTHORS View 1 chunk +0 lines, -10 lines 0 comments Download
M CONTRIBUTORS View 1 chunk +0 lines, -10 lines 0 comments Download
A codereview.cfg View 1 chunk +1 line, -0 lines 0 comments Download
R lib/codereview/codereview.cfg View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 3
r
Hello rsc (cc: nigeltao), I'd like you to review this change to https://code.google.com/p/ogle
10 years, 2 months ago (2014-02-10 21:53:05 UTC) #1
rsc
LGTM
10 years, 2 months ago (2014-02-10 21:57:22 UTC) #2
r
10 years, 2 months ago (2014-02-10 22:15:35 UTC) #3
*** Submitted as https://code.google.com/p/ogle/source/detail?r=e53db6031eee ***

ogle: fix A+C so we don't need to maintain contributors
Put coderefview.cfg in the root and copy it, plus A+C from go.tools.
Drop the defaultcc: line from codereview.cfg, at least for now.

LGTM=rsc
R=rsc
CC=nigeltao
https://codereview.appspot.com/61690044
Sign in to reply to this message.

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