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

Issue 94520045: Fix LP 1320138

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 10 months ago by dfc
Modified:
9 years, 10 months ago
Reviewers:
mp+219976, fwereade
Visibility:
Public.

Description

Fix LP 1320138 Constraints appear to be a set, not a list, so compare them in an unordered manner. https://code.launchpad.net/~dave-cheney/juju-core/174-fix-lp-1320138/+merge/219976 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M constraints/validation_test.go View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2
dfc
Please take a look.
9 years, 10 months ago (2014-05-19 02:55:30 UTC) #1
fwereade
9 years, 10 months ago (2014-05-19 09:17:38 UTC) #2
LGTM
Sign in to reply to this message.

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