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

Issue 328110043: [dfvfs] Changes for pylint 1.7 upgrade (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 8 months ago by Joachim Metz
Modified:
6 years, 8 months ago
Reviewers:
onager
CC:
kiddi, log2timeline-dev_googlegroups.com
Visibility:
Public.

Description

[dfvfs] Changes for pylint 1.7 upgrade

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+380 lines, -214 lines) Patch
M dfvfs/encoding/manager.py View 1 chunk +3 lines, -3 lines 0 comments Download
M dfvfs/encryption/manager.py View 1 chunk +2 lines, -2 lines 0 comments Download
M dfvfs/file_io/compressed_stream_io.py View 3 chunks +3 lines, -7 lines 0 comments Download
M dfvfs/file_io/encoded_stream_io.py View 2 chunks +3 lines, -7 lines 0 comments Download
M dfvfs/file_io/encrypted_stream_io.py View 2 chunks +3 lines, -7 lines 0 comments Download
M dfvfs/file_io/file_io.py View 1 chunk +2 lines, -2 lines 0 comments Download
M dfvfs/file_io/file_object_io.py View 2 chunks +2 lines, -6 lines 0 comments Download
M dfvfs/file_io/sqlite_blob_file_io.py View 1 chunk +1 line, -1 line 0 comments Download
M dfvfs/helpers/volume_scanner.py View 1 chunk +1 line, -1 line 0 comments Download
M dfvfs/lib/py2to3.py View 1 chunk +2 lines, -0 lines 0 comments Download
M dfvfs/serializer/json_serializer.py View 2 chunks +3 lines, -1 line 0 comments Download
M dfvfs/vfs/ntfs_file_entry.py View 2 chunks +6 lines, -10 lines 0 comments Download
M dfvfs/vfs/sqlite_blob_file_entry.py View 1 chunk +2 lines, -2 lines 0 comments Download
M dfvfs/vfs/sqlite_blob_file_system.py View 1 chunk +3 lines, -3 lines 0 comments Download
M dfvfs/vfs/tsk_file_entry.py View 3 chunks +9 lines, -16 lines 0 comments Download
M setup.py View 1 chunk +4 lines, -2 lines 0 comments Download
M utils/dependencies.py View 1 chunk +2 lines, -0 lines 0 comments Download
M utils/pylintrc View 4 chunks +329 lines, -144 lines 0 comments Download

Messages

Total messages: 4
Joachim Metz
6 years, 8 months ago (2017-07-25 06:56:08 UTC) #1
Joachim Metz
This CL was merged and committed with https://codereview.appspot.com/328120043/ via github. Please still have a look ...
6 years, 8 months ago (2017-07-27 06:14:08 UTC) #2
onager
LGTM
6 years, 8 months ago (2017-08-04 14:00:31 UTC) #3
Joachim Metz
6 years, 8 months ago (2017-08-05 05:30:48 UTC) #4
Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: python ./utils/review.py close pylint
Sign in to reply to this message.

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