{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/11934391","html_url":"https://github.com/jirikuncar/invenio/commit/32163a64ad1c47e1c4ea99e1b8a12cd5a9b00654#commitcomment-11934391","id":11934391,"user":{"login":"jirikuncar","id":189798,"avatar_url":"https://avatars.githubusercontent.com/u/189798?v=3","gravatar_id":"","url":"https://api.github.com/users/jirikuncar","html_url":"https://github.com/jirikuncar","followers_url":"https://api.github.com/users/jirikuncar/followers","following_url":"https://api.github.com/users/jirikuncar/following{/other_user}","gists_url":"https://api.github.com/users/jirikuncar/gists{/gist_id}","starred_url":"https://api.github.com/users/jirikuncar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jirikuncar/subscriptions","organizations_url":"https://api.github.com/users/jirikuncar/orgs","repos_url":"https://api.github.com/users/jirikuncar/repos","events_url":"https://api.github.com/users/jirikuncar/events{/privacy}","received_events_url":"https://api.github.com/users/jirikuncar/received_events","type":"User","site_admin":false},"position":0,"line":null,"path":"invenio/modules/encoder/batch_engine.py","commit_id":"32163a64ad1c47e1c4ea99e1b8a12cd5a9b00654","created_at":"2015-06-30T15:23:27Z","updated_at":"2015-06-30T15:23:27Z","body":"1: D400 First line should end with a period (not 'e')\n4: L102 copyright year is outdated, expected 2015 but got 2011\n23:1: I100 Imports statements are in the wrong order. from pprint should be before from string\n24:1: I100 Imports statements are in the wrong order. import os should be before from pprint\n29:1: I101 Imported names are in the wrong order. Should be task_low_level_submission, task_update_progress, write_message\n30:30: E126 continuation line over-indented for hanging indent\n34:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from invenio.legacy.bibsched.bibtask\n35:1: I101 Imported names are in the wrong order. Should be assure_quality, encode_video\n38:41: E126 continuation line over-indented for hanging indent\n42:1: I100 Imports statements are in the wrong order. from invenio.modules.encoder.metadata should be before from invenio.modules.encoder.utils\n43:41: E126 continuation line over-indented for hanging indent\n45:1: I101 Imported names are in the wrong order. Should be chose2, generate_timestamp, getval\n45:1: F999 redefinition of unused 'chose2' from line 41\n46:1: I101 Imported names are in the wrong order. Should be CFG_BIBENCODE_ASPECT_RATIO_MARC_FIELD, CFG_BIBENCODE_DAEMON_DIR_NEWJOBS, CFG_BIBENCODE_PBCORE_MARC_XSLT\n46:1: I100 Imports statements are in the wrong order. from invenio.modules.encoder.config should be before from invenio.modules.encoder.utils\n47:39: E126 continuation line over-indented for hanging indent\n51:1: I100 Imports statements are in the wrong order. from invenio.ext.email should be before from invenio.modules.encoder.config\n52:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.ext.email\n56:1: I100 Imports statements are in the wrong order. import invenio.config should be before from invenio.utils.json\n66:1: E302 expected 2 blank lines, found 1\n66: D401 First line should be in imperative mood ('Send', not 'Sends')\n66: D400 First line should end with a period (not 'g')\n66: D205 1 blank line required between summary line and description (found 0)\n77:13: E128 continuation line under-indented for visual indent\n78:13: E128 continuation line under-indented for visual indent\n79:13: E128 continuation line under-indented for visual indent\n80:13: E128 continuation line under-indented for visual indent\n81:13: E128 continuation line under-indented for visual indent\n83:14: E128 continuation line under-indented for visual indent\n91:1: E302 expected 2 blank lines, found 1\n91: D401 First line should be in imperative mood ('Send', not 'Sends')\n91: D205 1 blank line required between summary line and description (found 0)\n109:13: E128 continuation line under-indented for visual indent\n110:13: E128 continuation line under-indented for visual indent\n119:13: E128 continuation line under-indented for visual indent\n120:13: E128 continuation line under-indented for visual indent\n122:14: E128 continuation line under-indented for visual indent\n123:14: E128 continuation line under-indented for visual indent\n124:14: E128 continuation line under-indented for visual indent\n127:14: E128 continuation line under-indented for visual indent\n128:14: E128 continuation line under-indented for visual indent\n129:14: E128 continuation line under-indented for visual indent\n137:1: E302 expected 2 blank lines, found 1\n137: D401 First line should be in imperative mood ('Send', not 'Sends')\n137: D205 1 blank line required between summary line and description (found 0)\n153:13: E128 continuation line under-indented for visual indent\n154:13: E126 continuation line over-indented for hanging indent\n160:13: E128 continuation line under-indented for visual indent\n161:13: E128 continuation line under-indented for visual indent\n163:14: E128 continuation line under-indented for visual indent\n164:14: E128 continuation line under-indented for visual indent\n165:14: E128 continuation line under-indented for visual indent\n168:14: E128 continuation line under-indented for visual indent\n169:14: E128 continuation line under-indented for visual indent\n170:14: E128 continuation line under-indented for visual indent\n178:1: E302 expected 2 blank lines, found 1\n178: D210 No whitespaces allowed surrounding docstring text\n178: D401 First line should be in imperative mood ('Generate', not 'Generates')\n178: D205 1 blank line required between summary line and description (found 0)\n178: D208 Docstring is over-indented\n189:1: E302 expected 2 blank lines, found 1\n189: D210 No whitespaces allowed surrounding docstring text\n189: D401 First line should be in imperative mood ('Store', not 'Stores')\n189: D400 First line should end with a period (not 's')\n189: D205 1 blank line required between summary line and description (found 0)\n189: D208 Docstring is over-indented\n198:1: E302 expected 2 blank lines, found 1\n198: D401 First line should be in imperative mood ('Remove', not 'Removes')\n198: D400 First line should end with a period (not 'r')\n198: D205 1 blank line required between summary line and description (found 0)\n222:25: E128 continuation line under-indented for visual indent\n223:25: E128 continuation line under-indented for visual indent\n224:25: E128 continuation line under-indented for visual indent\n225:25: E128 continuation line under-indented for visual indent\n239:1: E302 expected 2 blank lines, found 1\n239: D210 No whitespaces allowed surrounding docstring text\n239: D401 First line should be in imperative mood ('Create', not 'Creates')\n239: D400 First line should end with a period (not 'n')\n239: D205 1 blank line required between summary line and description (found 0)\n240:29: E126 continuation line over-indented for hanging indent\n256:1: E302 expected 2 blank lines, found 1\n256: D210 No whitespaces allowed surrounding docstring text\n256: D401 First line should be in imperative mood ('Create', not 'Creates')\n256: D400 First line should end with a period (not 'a')\n256: D205 1 blank line required between summary line and description (found 0)\n274:1: E302 expected 2 blank lines, found 1\n274: D210 No whitespaces allowed surrounding docstring text\n274: D401 First line should be in imperative mood ('Create', not 'Creates')\n274: D400 First line should end with a period (not 's')\n274: D205 1 blank line required between summary line and description (found 0)\n297:1: E302 expected 2 blank lines, found 1\n297: D210 No whitespaces allowed surrounding docstring text\n297: D401 First line should be in imperative mood ('Create', not 'Creates')\n297: D400 First line should end with a period (not 'y')\n297: D205 1 blank line required between summary line and description (found 0)\n298:21: E126 continuation line over-indented for hanging indent\n320:1: E302 expected 2 blank lines, found 1\n320: D210 No whitespaces allowed surrounding docstring text\n320: D401 First line should be in imperative mood ('Check', not 'Checks')\n320: D400 First line should end with a period (not 'y')\n320: D205 1 blank line required between summary line and description (found 0)\n326: D210 No whitespaces allowed surrounding docstring text\n326: D401 First line should be in imperative mood ('Sanitize', not 'Sanitizes')\n326: D400 First line should end with a period (not 's')\n326: D200 One-line docstring should fit on one line with quotes (found 2)\n355:5: E129 visually indented line with same indent as next logical line\n357:21: E128 continuation line under-indented for visual indent\n373:1: E302 expected 2 blank lines, found 1\n373: D210 No whitespaces allowed surrounding docstring text\n373: D401 First line should be in imperative mood ('Processe', not 'Processes')\n373: D400 First line should end with a period (not 'y')\n383: D210 No whitespaces allowed surrounding docstring text\n383: D401 First line should be in imperative mood ('Create', not 'Creates')\n383: D400 First line should end with a period (not 'd')\n383: D200 One-line docstring should fit on one line with quotes (found 2)\n386:36: E225 missing whitespace around operator\n400:5: E266 too many leading '#' for block comment\n403:5: E266 too many leading '#' for block comment\n406:5: E266 too many leading '#' for block comment\n416:5: E266 too many leading '#' for block comment\n431:21: E129 visually indented line with same indent as next logical line\n434:21: E266 too many leading '#' for block comment\n436:25: E266 too many leading '#' for block comment\n437:47: F821 undefined name 'get_fieldvalues'\n445:27: E128 continuation line under-indented for visual indent\n448:13: E266 too many leading '#' for block comment\n451:5: E266 too many leading '#' for block comment\n458:5: E266 too many leading '#' for block comment\n466:5: E266 too many leading '#' for block comment\n469:5: E266 too many leading '#' for block comment\n473:5: E266 too many leading '#' for block comment\n481:13: E266 too many leading '#' for block comment\n482:13: E266 too many leading '#' for block comment\n483:13: E266 too many leading '#' for block comment\n484:13: E266 too many leading '#' for block comment\n486:27: E128 continuation line under-indented for visual indent\n488:37: E126 continuation line over-indented for hanging indent\n491:13: E266 too many leading '#' for block comment\n492:13: E266 too many leading '#' for block comment\n496:21: E126 continuation line over-indented for hanging indent\n513:28: E128 continuation line under-indented for visual indent\n515:9: E266 too many leading '#' for block comment\n525:13: E266 too many leading '#' for block comment\n533:13: E266 too many leading '#' for block comment\n538:13: E266 too many leading '#' for block comment\n541:13: E266 too many leading '#' for block comment\n542:13: E266 too many leading '#' for block comment\n544:50: E126 continuation line over-indented for hanging indent\n551:13: E266 too many leading '#' for block comment\n553:18: E126 continuation line over-indented for hanging indent\n567:53: E261 at least two spaces before inline comment\n573:13: E266 too many leading '#' for block comment\n574:15: E271 multiple spaces after keyword\n575:17: E266 too many leading '#' for block comment\n590:47: E127 continuation line over-indented for visual indent\n616:25: W503 line break before binary operator\n617:25: E128 continuation line under-indented for visual indent\n619:28: E128 continuation line under-indented for visual indent\n620:28: E128 continuation line under-indented for visual indent\n621:28: E126 continuation line over-indented for hanging indent\n631:13: E266 too many leading '#' for block comment\n633:17: E266 too many leading '#' for block comment\n634:17: E266 too many leading '#' for block comment\n637:21: E266 too many leading '#' for block comment\n638:21: E266 too many leading '#' for block comment\n641:21: E266 too many leading '#' for block comment\n644:21: E266 too many leading '#' for block comment\n648:21: E266 too many leading '#' for block comment\n652:21: E266 too many leading '#' for block comment\n653:21: E266 too many leading '#' for block comment\n657:35: E128 continuation line under-indented for visual indent\n660:37: E126 continuation line over-indented for hanging indent\n665:13: E266 too many leading '#' for block comment\n677:5: E266 too many leading '#' for block comment\n681:5: E266 too many leading '#' for block comment\n686:9: E266 too many leading '#' for block comment\n699:44: E251 unexpected spaces around keyword / parameter equals\n699:46: E251 unexpected spaces around keyword / parameter equals\n700:50: E251 unexpected spaces around keyword / parameter equals\n700:52: E251 unexpected spaces around keyword / parameter equals\n701:49: E251 unexpected spaces around keyword / parameter equals\n701:51: E251 unexpected spaces around keyword / parameter equals\n734:5: E266 too many leading '#' for block comment\n744:55: E721 do not compare types, use 'isinstance()'\n744:69: E202 whitespace before ')'\n754:32: E128 continuation line under-indented for visual indent\n755:32: E128 continuation line under-indented for visual indent\n756:32: E128 continuation line under-indented for visual indent\n758:1: W391 blank line at end of file"}}