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

Issue 29350277: Issue 4375 - Updated jshydra dependency for Python 3 support (Closed)

Created:
Aug. 29, 2016, 9:23 a.m. by Sebastian Noack
Modified:
Aug. 31, 2016, 8:45 a.m.
Reviewers:
Vasily Kuznetsov
Visibility:
Public.

Description

Issue 4375 - Updated jshydra dependency for Python 3 support

Patch Set 1 #

Total comments: 1

Patch Set 2 : Update revisions #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M dependencies View 1 1 chunk +1 line, -1 line 0 comments Download
M packagerChrome.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
Sebastian Noack
https://codereview.adblockplus.org/29350277/diff/29350278/dependencies File dependencies (right): https://codereview.adblockplus.org/29350277/diff/29350278/dependencies#newcode2 dependencies:2: jshydra = hg:file:///home/sebastian/src/jshydra@615baa23f4e6 Temporary hack to test the changes ...
Aug. 29, 2016, 9:25 a.m. (2016-08-29 09:25:26 UTC) #1
Vasily Kuznetsov
On 2016/08/29 09:25:26, Sebastian Noack wrote: > https://codereview.adblockplus.org/29350277/diff/29350278/dependencies > File dependencies (right): > > https://codereview.adblockplus.org/29350277/diff/29350278/dependencies#newcode2 ...
Aug. 29, 2016, 3:27 p.m. (2016-08-29 15:27:17 UTC) #2
Sebastian Noack
Aug. 30, 2016, 1:56 p.m. (2016-08-30 13:56:20 UTC) #3
Vasily Kuznetsov
Aug. 30, 2016, 2:15 p.m. (2016-08-30 14:15:24 UTC) #4
On 2016/08/30 13:56:20, Sebastian Noack wrote:

LGTM

Powered by Google App Engine
This is Rietveld