Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29720593: ABB: Noissue - Carthage bump (Closed)

Created:
March 12, 2018, 12:32 p.m. by a.shephard
Modified:
March 14, 2018, 9:17 a.m.
Reviewers:
d108, dean
Visibility:
Public.

Description

ABB: Noissue - Carthage bump

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M Cartfile.resolved View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
a.shephard
bumped Carthage dependency versions on Carthage.resolved file
March 12, 2018, 12:33 p.m. (2018-03-12 12:33:31 UTC) #1
dean
On 2018/03/12 12:33:31, a.shephard wrote: > bumped Carthage dependency versions on Carthage.resolved file LGTM
March 13, 2018, 10:10 a.m. (2018-03-13 10:10:39 UTC) #2
d108
On 2018/03/13 10:10:39, dean wrote: > On 2018/03/12 12:33:31, a.shephard wrote: > > bumped Carthage ...
March 14, 2018, 1:56 a.m. (2018-03-14 01:56:28 UTC) #3
a.shephard
March 14, 2018, 9:01 a.m. (2018-03-14 09:01:53 UTC) #4
On 2018/03/14 01:56:28, d108 wrote:
> On 2018/03/13 10:10:39, dean wrote:
> > On 2018/03/12 12:33:31, a.shephard wrote:
> > > bumped Carthage dependency versions on Carthage.resolved file
> > 
> > LGTM
> 
> LGTM but the dependencies should be hosted by adblockplus

Agreed. We can do this as part of a separate issue once the dependencies have
been moved (either GH or GL).

Powered by Google App Engine
This is Rietveld