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

Issue 11745043: Trivial import testing prefix.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by thumper
Modified:
10 years, 9 months ago
Reviewers:
mp+176543, dave, wallyworld
Visibility:
Public.

Description

Trivial import testing prefix. gc and jc prefix for imports. https://code.launchpad.net/~thumper/juju-core/state-gc-prefix/+merge/176543 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+520 lines, -518 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M state/state_test.go View 49 chunks +500 lines, -500 lines 0 comments Download
M state/tools_test.go View 3 chunks +18 lines, -18 lines 0 comments Download

Messages

Total messages: 3
thumper
Please take a look.
10 years, 9 months ago (2013-07-23 23:00:07 UTC) #1
wallyworld
LGTM
10 years, 9 months ago (2013-07-23 23:57:14 UTC) #2
dave_cheney.net
10 years, 9 months ago (2013-07-24 00:20:13 UTC) #3
On 2013/07/23 23:57:14, wallyworld wrote:
> LGTM

LGTM. Trivial change is trivial.
Sign in to reply to this message.

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