{"comment":{"url":"https://api.github.com/repos/jirikuncar/invenio/comments/10138168","html_url":"https://github.com/jirikuncar/invenio/commit/5e9ee89ecf8afefbc70b5139d6d5d1496f376ca8#commitcomment-10138168","id":10138168,"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/legacy/bibcheck/task.py","commit_id":"5e9ee89ecf8afefbc70b5139d6d5d1496f376ca8","created_at":"2015-03-11T09:37:56Z","updated_at":"2015-03-11T09:37:56Z","body":"1: D400 First line should end with '.', not 'k'\n4: L102 copyright year is outdated, expected 2015 but got 2014\n26:1: I100 Imports statements are in the wrong order. import getopt should be before import sys\n29:1: I100 Imports statements are in the wrong order. import time should be before import traceback\n30:1: I100 Imports statements are in the wrong order. import inspect should be before import time\n34:1: I100 Imports statements are in the wrong order. from ConfigParser should be before from collections\n37:1: I101 Imported names are in the wrong order. Should be get_modified_records_since, task_get_option, task_get_task_param, task_has_option, task_init, task_low_level_submission, task_set_option, task_sleep_now_if_required, task_update_progress, write_message\n37:1: I201 Missing newline before sections or imports.\n47:1: I101 Imported names are in the wrong order. Should be CFG_CERN_SITE, CFG_ETCDIR, CFG_PYLIBDIR, CFG_SITE_URL, CFG_TMPSHAREDDIR, CFG_VERSION\n47:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibsched.bibtask\n54:1: I101 Imported names are in the wrong order. Should be perform_request_search, search_pattern, search_unit_in_bibxxx\n58:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibedit.utils should be before from invenio.legacy.search_engine\n59:1: I101 Imported names are in the wrong order. Should be record_add_field, record_xml_output\n60:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.bibrecord\n60:1: I201 Missing newline before sections or imports.\n61:1: I201 Missing newline before sections or imports.\n62:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.api should be before from invenio.legacy.dbquery\n64:1: I100 Imports statements are in the wrong order. from invenio.utils.json should be before from invenio.utils.shell\n68:1: E302 expected 2 blank lines, found 1\n68: D290 White spaces around docstring should be trimmed.\n68: D203 Expected 1 blank line *before* class docstring, found 0\n68: D204 Expected 1 blank line *after* class docstring, found 0\n68: D400 First line should end with '.', not 'n'\n70: D102 Docstring missing\n72:13: E128 continuation line under-indented for visual indent\n75: D290 White spaces around docstring should be trimmed.\n75: D203 Expected 1 blank line *before* class docstring, found 0\n75: D204 Expected 1 blank line *after* class docstring, found 0\n75: D400 First line should end with '.', not 'n'\n77: D102 Docstring missing\n88: D401 First line should be imperative: 'Iterate', not 'Iterates'\n149:55: E261 at least two spaces before inline comment\n163: D290 White spaces around docstring should be trimmed.\n163: D400 First line should end with '.', not 'n'\n169:57: E261 at least two spaces before inline comment\n175: D290 White spaces around docstring should be trimmed.\n175: D400 First line should end with '.', not 'e'\n182: D401 First line should be imperative: 'Change', not 'Changes'\n182: D400 First line should end with '.', not 'e'\n182: D205 Blank line missing between one-line summary and description\n201:25: E128 continuation line under-indented for visual indent\n204: D400 First line should end with '.', not 'd'\n204: D200 One-line docstring should not occupy 3 lines\n216: D290 White spaces around docstring should be trimmed.\n216: D400 First line should end with '.', not 'd'\n222: D290 White spaces around docstring should be trimmed.\n222: D400 First line should end with '.', not 'd'\n226:13: E128 continuation line under-indented for visual indent\n228: D290 White spaces around docstring should be trimmed.\n228: D400 First line should end with '.', not 'd'\n231:17: E128 continuation line under-indented for visual indent\n237: D290 White spaces around docstring should be trimmed.\n237: D400 First line should end with '.', not 'd'\n240:17: E128 continuation line under-indented for visual indent\n244: D290 White spaces around docstring should be trimmed.\n244: D400 First line should end with '.', not 'd'\n248:17: E128 continuation line under-indented for visual indent\n250: D290 White spaces around docstring should be trimmed.\n250: D400 First line should end with '.', not 't'\n254: D401 First line should be imperative: 'Yield', not 'Yields'\n254: D400 First line should end with '.', not 'g'\n265:5: E129 visually indented line with same indent as next logical line\n271: D102 Docstring missing\n276: D290 White spaces around docstring should be trimmed.\n276: D202 No blank lines allowed *after* function docstring, found 1\n276: D400 First line should end with '.', not 'k'\n301:1: E302 expected 2 blank lines, found 1\n341:25: E128 continuation line under-indented for visual indent\n367:5: E266 too many leading '#' for block comment\n379:1: E302 expected 2 blank lines, found 1\n379: D290 White spaces around docstring should be trimmed.\n379: D202 No blank lines allowed *after* function docstring, found 1\n379: D400 First line should end with '.', not 'g'\n422: D290 White spaces around docstring should be trimmed.\n422: D202 No blank lines allowed *after* function docstring, found 1\n422: D400 First line should end with '.', not 'd'\n448:1: E302 expected 2 blank lines, found 1\n448: D400 First line should end with '.', not 'e'\n448: D200 One-line docstring should not occupy 3 lines\n458:1: E302 expected 2 blank lines, found 1\n458: D400 First line should end with '.', not 'e'\n458: D200 One-line docstring should not occupy 3 lines\n468:1: E302 expected 2 blank lines, found 1\n468: D400 First line should end with '.', not 'e'\n468: D205 Blank line missing between one-line summary and description\n481: D202 No blank lines allowed *after* function docstring, found 1\n481: D400 First line should end with '.', not 'd'\n481: D205 Blank line missing between one-line summary and description\n493:20: E261 at least two spaces before inline comment\n498: D400 First line should end with '.', not 'e'\n498: D205 Blank line missing between one-line summary and description\n505:1: E302 expected 2 blank lines, found 1\n528:1: E302 expected 2 blank lines, found 1\n528: D290 White spaces around docstring should be trimmed.\n528: D400 First line should end with '.', not 'e'\n537: D290 White spaces around docstring should be trimmed.\n537: D400 First line should end with '.', not 's'\n554:47: E127 continuation line over-indented for visual indent\n587: D400 First line should end with '.', not 's'\n587: D200 One-line docstring should not occupy 3 lines\n611: D401 First line should be imperative: 'Generate', not 'Generates'\n652:16: E713 test for membership should be 'not in'\n662: D400 First line should end with '.', not 'e'\n662: D205 Blank line missing between one-line summary and description\n675:1: E302 expected 2 blank lines, found 1\n675: D400 First line should end with '.', not 'a'\n675: D205 Blank line missing between one-line summary and description\n730: D401 First line should be imperative: 'Print', not 'Prints'\n730: D400 First line should end with '.', not 't'\n732:12: E221 multiple spaces before operator\n748: D401 First line should be imperative: 'Print', not 'Prints'\n748: D400 First line should end with '.', not 't'\n762: D401 First line should be imperative: 'Construct', not 'Constructs'\n801:36: E127 continuation line over-indented for visual indent\n820:19: E128 continuation line under-indented for visual indent\n821:19: E128 continuation line under-indented for visual indent\n824:1: W391 blank line at end of file"}}