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

Issue 29750572: Noissue - Removed PIL from depdendencies in REAMDE, its not longer required by buildtools (Closed)

Created:
April 12, 2018, 12:29 p.m. by Sebastian Noack
Modified:
April 14, 2018, 2:14 a.m.
Reviewers:
kzar
CC:
tlucas
Visibility:
Public.

Description

Noissue - Removed PIL from README, its not longer required by buildtools

Patch Set 1 #

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

Messages

Total messages: 3
Sebastian Noack
April 12, 2018, 12:30 p.m. (2018-04-12 12:30:13 UTC) #1
kzar
Nit: Commit message is too long and has two typos Assuming it's true that we ...
April 13, 2018, 3:48 p.m. (2018-04-13 15:48:18 UTC) #2
Sebastian Noack
April 14, 2018, 2:13 a.m. (2018-04-14 02:13:12 UTC) #3
On 2018/04/13 15:48:18, kzar wrote:
> Nit: Commit message is too long and has two typos

Fixed.

> Assuming it's true that we don't need PIL any more then this LGTM

For reference, https://hg.adblockplus.org/buildtools/rev/20d6b0b276a9#l5.30

Powered by Google App Engine
This is Rietveld