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

Issue 29348920: Issue 3767 - Update README to reflect build changes in 44.0.2 (Closed)

Created:
July 29, 2016, 10:10 p.m. by diegocarloslima
Modified:
Nov. 2, 2016, 2:42 p.m.
Reviewers:
anton, Felix Dahlke
CC:
René Jeschke
Visibility:
Public.

Description

Issue 3767 - Update README to reflect build changes in 44.0.2

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -14 lines) Patch
M README.md View 3 chunks +4 lines, -14 lines 1 comment Download

Messages

Total messages: 5
diegocarloslima
July 29, 2016, 10:10 p.m. (2016-07-29 22:10:58 UTC) #1
anton
https://codereview.adblockplus.org/29348920/diff/29348921/README.md File README.md (left): https://codereview.adblockplus.org/29348920/diff/29348921/README.md#oldcode25 README.md:25: (Follow-up issue: https://issues.adblockplus.org/ticket/2490) currently it's also in reviewing state ...
Sept. 12, 2016, 11:29 a.m. (2016-09-12 11:29:26 UTC) #2
diegocarloslima
On 2016/09/12 11:29:26, anton wrote: > https://codereview.adblockplus.org/29348920/diff/29348921/README.md > File README.md (left): > > https://codereview.adblockplus.org/29348920/diff/29348921/README.md#oldcode25 > ...
Oct. 20, 2016, 10:17 a.m. (2016-10-20 10:17:19 UTC) #3
Felix Dahlke
LGTM, though I guess the README change regarding #2490 makes more sense along with the ...
Oct. 25, 2016, 3:18 p.m. (2016-10-25 15:18:20 UTC) #4
anton
Nov. 2, 2016, 2:03 p.m. (2016-11-02 14:03:17 UTC) #5
On 2016/10/20 10:17:19, diegocarloslima wrote:
> On 2016/09/12 11:29:26, anton wrote:
> > https://codereview.adblockplus.org/29348920/diff/29348921/README.md
> > File README.md (left):
> > 
> >
https://codereview.adblockplus.org/29348920/diff/29348921/README.md#oldcode25
> > README.md:25: (Follow-up issue: https://issues.adblockplus.org/ticket/2490)
> > currently it's also in reviewing state and i'd prefer this issue (and review
> > https://codereview.adblockplus.org/29342994/) to be closed too. Please
> consider
> > adding code reviews dependencies too (like #depends on .. for this one)
> 
> Added dependency to the ticket

LGTM

Powered by Google App Engine
This is Rietveld