1: D100 Missing docstring in public module
23:1: I101 Imported names are in the wrong order. Should be cPickle, calendar, cgi, commands, datetime, os, random, time
23:16: E401 multiple imports on one line
23:1: E402 module level import not at top of file
24:1: E402 module level import not at top of file
25:1: I101 Imported names are in the wrong order. Should be CFG_BIBCIRCULATION_ILL_STATUS_CANCELLED, CFG_BIBCIRCULATION_ITEM_STATUS_CANCELLED, CFG_BIBCIRCULATION_ITEM_STATUS_CLAIMED, CFG_BIBCIRCULATION_ITEM_STATUS_IN_PROCESS, CFG_BIBCIRCULATION_ITEM_STATUS_NOT_ARRIVED, CFG_BIBCIRCULATION_ITEM_STATUS_ON_LOAN, CFG_BIBCIRCULATION_ITEM_STATUS_ON_ORDER, CFG_BIBCIRCULATION_ITEM_STATUS_ON_SHELF, CFG_BIBCIRCULATION_ITEM_STATUS_OPTIONAL, CFG_BIBCIRCULATION_REQUEST_STATUS_DONE, CFG_BINDIR, CFG_CERN_SITE, CFG_SITE_NAME, CFG_SITE_URL, CFG_TMPDIR
25:1: I201 Missing newline before sections or imports.
25:1: E402 module level import not at top of file
40:1: E402 module level import not at top of file
41:1: E402 module level import not at top of file
42:1: I101 Imported names are in the wrong order. Should be get_collection_reclist, get_most_popular_field_values, perform_request_search, search_pattern
42:1: I100 Imports statements are in the wrong order. from invenio.legacy.search_engine should be before from invenio.utils.url
42:1: E402 module level import not at top of file
46:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine
46:1: E402 module level import not at top of file
47:1: E402 module level import not at top of file
49:1: I101 Imported names are in the wrong order. Should be book_information_from_MARC, book_title_from_MARC
49:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcirculation.utils should be before from invenio.legacy.dbquery
49:1: E402 module level import not at top of file
51:1: I101 Imported names are in the wrong order. Should be get_borrower_data, get_id_bibrec
51:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcirculation.db_layer should be before from invenio.legacy.bibcirculation.utils
51:1: E402 module level import not at top of file
54:1: I101 Imported names are in the wrong order. Should be convert_datestruct_to_dategui, convert_datetext_to_datestruct
54:1: E402 module level import not at top of file
55:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibsched.bibtask should be before from invenio.utils.date
55:1: E402 module level import not at top of file
58:38: E261 at least two spaces before inline comment
63:1: E302 expected 2 blank lines, found 1
63: D401 First line should be in imperative mood ('Return', not 'Returns')
63: D400 First line should end with a period (not 'r')
63: D205 1 blank line required between summary line and description (found 0)
87:25: E128 continuation line under-indented for visual indent
91:29: E128 continuation line under-indented for visual indent
91:61: E251 unexpected spaces around keyword / parameter equals
92:29: E128 continuation line under-indented for visual indent
93:29: E128 continuation line under-indented for visual indent
104:27: E128 continuation line under-indented for visual indent
107: D401 First line should be in imperative mood ('Return', not 'Returns')
107: D202 No blank lines allowed after function docstring (found 1)
129:29: E128 continuation line under-indented for visual indent
130:29: E128 continuation line under-indented for visual indent
144: D210 No whitespaces allowed surrounding docstring text
144: D209 Multi-line docstring closing quotes should be on a separate line
144: D400 First line should end with a period (not 's')
144: D205 1 blank line required between summary line and description (found 0)
153:29: E128 continuation line under-indented for visual indent
154:29: E126 continuation line over-indented for hanging indent
160:27: E128 continuation line under-indented for visual indent
163: D401 First line should be in imperative mood ('Return', not 'Returns')
163: D202 No blank lines allowed after function docstring (found 1)
163: D400 First line should end with a period (not 't')
163: D205 1 blank line required between summary line and description (found 0)
182:17: E128 continuation line under-indented for visual indent
183:29: E128 continuation line under-indented for visual indent
186: D401 First line should be in imperative mood ('Return', not 'Returns')
186: D400 First line should end with a period (not 't')
186: D205 1 blank line required between summary line and description (found 0)
208:13: E128 continuation line under-indented for visual indent
211:13: E128 continuation line under-indented for visual indent
211:44: E251 unexpected spaces around keyword / parameter equals
212:13: E128 continuation line under-indented for visual indent
216: D401 First line should be in imperative mood ('Return', not 'Returns')
216: D400 First line should end with a period (not 'n')
216: D205 1 blank line required between summary line and description (found 0)
236:21: E128 continuation line under-indented for visual indent
237:21: E128 continuation line under-indented for visual indent
241:21: E128 continuation line under-indented for visual indent
242:21: E128 continuation line under-indented for visual indent
246:25: E128 continuation line under-indented for visual indent
248:25: E128 continuation line under-indented for visual indent
257: D401 First line should be in imperative mood ('Return', not 'Returns')
257: D400 First line should end with a period (not 't')
257: D205 1 blank line required between summary line and description (found 0)
280:17: E128 continuation line under-indented for visual indent
300:27: E128 continuation line under-indented for visual indent
303: D401 First line should be in imperative mood ('Return', not 'Returns')
303: D400 First line should end with a period (not 't')
303: D205 1 blank line required between summary line and description (found 0)
321:13: E128 continuation line under-indented for visual indent
324: D401 First line should be in imperative mood ('Return', not 'Returns')
324: D400 First line should end with a period (not 'r')
324: D205 1 blank line required between summary line and description (found 0)
346:17: E128 continuation line under-indented for visual indent
350:17: E128 continuation line under-indented for visual indent
357: D400 First line should end with a period (not ':')
357: D205 1 blank line required between summary line and description (found 0)
428:37: E126 continuation line over-indented for hanging indent
429:10: E261 at least two spaces before inline comment
466: D400 First line should end with a period (not ':')
466: D205 1 blank line required between summary line and description (found 0)
575:62: E502 the backslash is redundant between brackets
576:23: E128 continuation line under-indented for visual indent
587:13: E128 continuation line under-indented for visual indent
588:13: E128 continuation line under-indented for visual indent
589:13: E128 continuation line under-indented for visual indent
590:13: E128 continuation line under-indented for visual indent
596:13: E128 continuation line under-indented for visual indent
597:13: E128 continuation line under-indented for visual indent
598:13: E128 continuation line under-indented for visual indent
599:13: E128 continuation line under-indented for visual indent
600:13: E128 continuation line under-indented for visual indent
605: D400 First line should end with a period (not ':')
605: D205 1 blank line required between summary line and description (found 0)
673: D400 First line should end with a period (not ':')
673: D205 1 blank line required between summary line and description (found 0)
699: D401 First line should be in imperative mood ('Return', not 'Returns')
699: D400 First line should end with a period (not 's')
717:17: E128 continuation line under-indented for visual indent
719:17: E128 continuation line under-indented for visual indent
723:17: E128 continuation line under-indented for visual indent
724:17: E128 continuation line under-indented for visual indent
728:25: E128 continuation line under-indented for visual indent
730:25: E128 continuation line under-indented for visual indent
739: D400 First line should end with a period (not ':')
739: D205 1 blank line required between summary line and description (found 0)
837: D400 First line should end with a period (not ':')
837: D205 1 blank line required between summary line and description (found 0)
866:21: E127 continuation line over-indented for visual indent
897: D401 First line should be in imperative mood ('Return', not 'Returns')
897: D400 First line should end with a period (not 't')
897: D205 1 blank line required between summary line and description (found 0)
943:19: E128 continuation line under-indented for visual indent
948:19: E128 continuation line under-indented for visual indent
948:66: E225 missing whitespace around operator
952:25: E128 continuation line under-indented for visual indent
954:25: E128 continuation line under-indented for visual indent
963: D400 First line should end with a period (not ':')
963: D205 1 blank line required between summary line and description (found 0)
1014: D202 No blank lines allowed after function docstring (found 1)
1014: D400 First line should end with a period (not ':')
1014: D205 1 blank line required between summary line and description (found 0)
1079: D400 First line should end with a period (not ':')
1079: D205 1 blank line required between summary line and description (found 0)
1122:57: E128 continuation line under-indented for visual indent
1137:9: E128 continuation line under-indented for visual indent
1140: D400 First line should end with a period (not ':')
1140: D205 1 blank line required between summary line and description (found 0)
1199: D400 First line should end with a period (not ':')
1199: D205 1 blank line required between summary line and description (found 0)
1235: D400 First line should end with a period (not ':')
1235: D205 1 blank line required between summary line and description (found 0)
1281:1: E302 expected 2 blank lines, found 1
1342: D400 First line should end with a period (not 'n')
1342: D205 1 blank line required between summary line and description (found 0)
1354:10: E261 at least two spaces before inline comment
1354:11: E262 inline comment should start with '# '
1376:83: E502 the backslash is redundant between brackets
1376:82: E211 whitespace before '('
1377:21: E128 continuation line under-indented for visual indent
1379:82: E502 the backslash is redundant between brackets
1379:81: E211 whitespace before '('
1380:21: E128 continuation line under-indented for visual indent
1382:82: E502 the backslash is redundant between brackets
1382:81: E211 whitespace before '('
1383:21: E128 continuation line under-indented for visual indent
1386:82: E502 the backslash is redundant between brackets
1386:81: E211 whitespace before '('
1387:21: E128 continuation line under-indented for visual indent
1394:1: E302 expected 2 blank lines, found 1
1394: D401 First line should be in imperative mood ('Build', not 'Builds')
1394: D400 First line should end with a period (not 'r')
1394: D205 1 blank line required between summary line and description (found 0)
1446:1: E302 expected 2 blank lines, found 1
1446: D205 1 blank line required between summary line and description (found 0)
1456:13: E127 continuation line over-indented for visual indent
1459:1: E302 expected 2 blank lines, found 1
1459: D401 First line should be in imperative mood ('Fetche', not 'Fetches')
1459: D400 First line should end with a period (not 't')
1459: D205 1 blank line required between summary line and description (found 0)
1471:15: E711 comparison to None should be 'if cond is None:'
1471:15: E225 missing whitespace around operator
1471:32: E225 missing whitespace around operator
1475:5: B901 blind except: statement
1493:1: E302 expected 2 blank lines, found 1
1493: D401 First line should be in imperative mood ('Return', not 'Returns')
1493: D205 1 blank line required between summary line and description (found 0)
1510:77: E502 the backslash is redundant between brackets
1511:25: E127 continuation line over-indented for visual indent
1515:81: E502 the backslash is redundant between brackets
1516:25: E127 continuation line over-indented for visual indent
1526:1: E302 expected 2 blank lines, found 1
1526: D401 First line should be in imperative mood ('Return', not 'Returns')
1526: D400 First line should end with a period (not 'n')
1526: D205 1 blank line required between summary line and description (found 0)
1560:33: E128 continuation line under-indented for visual indent
1561:28: E211 whitespace before '('
1567:33: E128 continuation line under-indented for visual indent
1568:28: E211 whitespace before '('
1576:28: E211 whitespace before '('
1585:33: E128 continuation line under-indented for visual indent
1590:77: E502 the backslash is redundant between brackets
1590:76: E211 whitespace before '('
1591:21: E128 continuation line under-indented for visual indent
1593:1: E302 expected 2 blank lines, found 1
1593: D400 First line should end with a period (not 't')
1593: D205 1 blank line required between summary line and description (found 0)
1606:13: E127 continuation line over-indented for visual indent
1608:1: E302 expected 2 blank lines, found 1
1608:33: E211 whitespace before '('
1608: D401 First line should be in imperative mood ('Return', not 'Returns')
1608: D205 1 blank line required between summary line and description (found 0)
1617:73: E502 the backslash is redundant between brackets
1617:72: E211 whitespace before '('
1618:21: E128 continuation line under-indented for visual indent
1621:1: E302 expected 2 blank lines, found 1
1621: D401 First line should be in imperative mood ('Return', not 'Returns')
1621: D400 First line should end with a period (not 'r')
1621: D205 1 blank line required between summary line and description (found 0)
1635:1: E303 too many blank lines (3)
1637:1: E302 expected 2 blank lines, found 3
1637: D401 First line should be in imperative mood ('Create', not 'Creates')
1637: D400 First line should end with a period (not 's')
1642: D210 No whitespaces allowed surrounding docstring text
1642: D401 First line should be in imperative mood ('Return', not 'Returns')
1642: D400 First line should end with a period (not 'g')
1647: D401 First line should be in imperative mood ('Return', not 'Returns')
1647: D400 First line should end with a period (not ')')
1652: D401 First line should be in imperative mood ('Return', not 'Returns')
1658: D210 No whitespaces allowed surrounding docstring text
1658: D401 First line should be in imperative mood ('Return', not 'Returns')
1658: D400 First line should end with a period (not 'g')
1664:1: E302 expected 2 blank lines, found 1
1664: D401 First line should be in imperative mood ('Return', not 'Returns')
1664: D400 First line should end with a period (not 'n')
1664: D205 1 blank line required between summary line and description (found 0)
1697:12: E713 test for membership should be 'not in'
1704:34: E127 continuation line over-indented for visual indent
1707:34: E127 continuation line over-indented for visual indent
1710:34: E127 continuation line over-indented for visual indent
1723: D400 First line should end with a period (not 'y')
1723: D205 1 blank line required between summary line and description (found 0)
1756:26: E127 continuation line over-indented for visual indent
1761:114: E261 at least two spaces before inline comment
1765:16: E713 test for membership should be 'not in'
1769:49: E502 the backslash is redundant between brackets
1770:42: E127 continuation line over-indented for visual indent
1772:48: E502 the backslash is redundant between brackets
1773:42: E127 continuation line over-indented for visual indent
1775:53: E502 the backslash is redundant between brackets
1776:42: E127 continuation line over-indented for visual indent
1790:21: E126 continuation line over-indented for hanging indent
1809: D400 First line should end with a period (not 'g')
1809: D205 1 blank line required between summary line and description (found 0)
1823: D400 First line should end with a period (not 'g')
1823: D205 1 blank line required between summary line and description (found 0)
1840:1: E302 expected 2 blank lines, found 1
1840: D401 First line should be in imperative mood ('Return', not 'Returns')
1840: D202 No blank lines allowed after function docstring (found 1)
1840: D400 First line should end with a period (not 'r')
1883: D401 First line should be in imperative mood ('Create', not 'Creates')
1883: D400 First line should end with a period (not 'd')
1883: D205 1 blank line required between summary line and description (found 0)
1897:5: B901 blind except: statement
1950:1: E302 expected 2 blank lines, found 1
1950: D401 First line should be in imperative mood ('Create', not 'Creates')
1978: D401 First line should be in imperative mood ('Create', not 'Creates')
1978: D400 First line should end with a period (not 't')
1989:21: E128 continuation line under-indented for visual indent
1997:21: E126 continuation line over-indented for hanging indent
2036:37: E128 continuation line under-indented for visual indent
2045: D401 First line should be in imperative mood ('Create', not 'Creates')
2045: D400 First line should end with a period (not 'y')
2093:35: E127 continuation line over-indented for visual indent
2117: D401 First line should be in imperative mood ('Create', not 'Creates')
2117: D400 First line should end with a period (not 'y')
2143:25: E128 continuation line under-indented for visual indent
2168:21: E128 continuation line under-indented for visual indent
2168:60: E225 missing whitespace around operator
2169:21: E126 continuation line over-indented for hanging indent
2207:18: E222 multiple spaces after operator
2225:9: E116 unexpected indentation (comment)
2243:9: E116 unexpected indentation (comment)
2307:62: E203 whitespace before ':'
2311: D210 No whitespaces allowed surrounding docstring text
2311: D401 First line should be in imperative mood ('Return', not 'Returns')
2311: D400 First line should end with a period (not 'a')
2326: D401 First line should be in imperative mood ('Create', not 'Creates')
2363: D401 First line should be in imperative mood ('Create', not 'Creates')
2386:46: E502 the backslash is redundant between brackets
2387:21: E128 continuation line under-indented for visual indent
2413:1: E302 expected 2 blank lines, found 1
2413: D401 First line should be in imperative mood ('Return', not 'Returns')
2413: D400 First line should end with a period (not 'y')
2419: D401 First line should be in imperative mood ('Return', not 'Returns')
2419: D400 First line should end with a period (not 'y')
2431: D401 First line should be in imperative mood ('Return', not 'Returns')
2431: D400 First line should end with a period (not 'y')
2439:14: E128 continuation line under-indented for visual indent
2444: D401 First line should be in imperative mood ('Return', not 'Returns')
2444: D400 First line should end with a period (not 'y')
2449: D401 First line should be in imperative mood ('Return', not 'Returns')
2449: D400 First line should end with a period (not 'y')
2454: D401 First line should be in imperative mood ('Return', not 'Returns')
2454: D400 First line should end with a period (not 'y')
2459: D401 First line should be in imperative mood ('Return', not 'Returns')
2459: D400 First line should end with a period (not 'y')
2466: D401 First line should be in imperative mood ('Return', not 'Returns')
2466: D400 First line should end with a period (not 'y')
2474: D401 First line should be in imperative mood ('Return', not 'Returns')
2474: D400 First line should end with a period (not 'y')
2482: D401 First line should be in imperative mood ('Return', not 'Returns')
2482: D400 First line should end with a period (not 'y')
2490: D401 First line should be in imperative mood ('Return', not 'Returns')
2490: D400 First line should end with a period (not 's')
2494:10: E128 continuation line under-indented for visual indent
2498:1: E302 expected 2 blank lines, found 1
2498: D401 First line should be in imperative mood ('Export', not 'Exports')
2511: D401 First line should be in imperative mood ('Export', not 'Exports')
2521:66: E502 the backslash is redundant between brackets
2522:18: E128 continuation line under-indented for visual indent
2526: D401 First line should be in imperative mood ('Export', not 'Exports')
2546:68: E502 the backslash is redundant between brackets
2556:1: E302 expected 2 blank lines, found 1
2556: D400 First line should end with a period (not 'a')
2556: D205 1 blank line required between summary line and description (found 0)
2565:64: E502 the backslash is redundant between brackets
2566:25: E127 continuation line over-indented for visual indent
2569: D400 First line should end with a period (not ',')
2569: D205 1 blank line required between summary line and description (found 0)
2624:13: E128 continuation line under-indented for visual indent
2625:13: E128 continuation line under-indented for visual indent
2626:13: E128 continuation line under-indented for visual indent
2627:13: E128 continuation line under-indented for visual indent
2628:13: E128 continuation line under-indented for visual indent
2629:13: E124 closing bracket does not match visual indentation
2629:13: E125 continuation line with same indent as next logical line
2645:13: E125 continuation line with same indent as next logical line
2651: D401 First line should be in imperative mood ('Return', not 'Returns')
2677:27: E128 continuation line under-indented for visual indent
2680: D401 First line should be in imperative mood ('Return', not 'Returns')
2680: D400 First line should end with a period (not ',')
2680: D205 1 blank line required between summary line and description (found 0)
2724:1: E302 expected 2 blank lines, found 1
2724: D401 First line should be in imperative mood ('Transform', not 'Transforms')
2724: D400 First line should end with a period (not 'e')
2724: D200 One-line docstring should fit on one line with quotes (found 3)
2731: D401 First line should be in imperative mood ('Run', not 'Runs')
2731: D400 First line should end with a period (not 's')
2731: D205 1 blank line required between summary line and description (found 0)
2745: D401 First line should be in imperative mood ('Return', not 'Returns')
2745: D400 First line should end with a period (not 'n')
2751: D401 First line should be in imperative mood ('Return', not 'Returns')
2751: D400 First line should end with a period (not 'm')
2760:13: E127 continuation line over-indented for visual indent
2763: D401 First line should be in imperative mood ('Return', not 'Returns')
2763: D400 First line should end with a period (not 'm')
2767:9: E125 continuation line with same indent as next logical line
2772: D401 First line should be in imperative mood ('Return', not 'Returns')
2772: D400 First line should end with a period (not 't')
2780: D401 First line should be in imperative mood ('Return', not 'Returns')
2780: D400 First line should end with a period (not 's')
2784:33: E713 test for membership should be 'not in'
2784:54: E261 at least two spaces before inline comment
2789: D401 First line should be in imperative mood ('Return', not 'Returns')
2789: D400 First line should end with a period (not 'm')
2797: D400 First line should end with a period (not 'e')
2797: D200 One-line docstring should fit on one line with quotes (found 3)
2809:1: E302 expected 2 blank lines, found 1
2815:1: E302 expected 2 blank lines, found 1
2820:1: E302 expected 2 blank lines, found 1
2827:1: E302 expected 2 blank lines, found 1
2833:1: E302 expected 2 blank lines, found 1
2846:1: E302 expected 2 blank lines, found 1
2851:37: E128 continuation line under-indented for visual indent
2853:10: E128 continuation line under-indented for visual indent
2854:10: E128 continuation line under-indented for visual indent
2855:10: E128 continuation line under-indented for visual indent
2856:10: E128 continuation line under-indented for visual indent
2857:10: E128 continuation line under-indented for visual indent
2858:10: E128 continuation line under-indented for visual indent


1: D210 No whitespaces allowed surrounding docstring text
27:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n
29:1: F401 'config' imported but unused
29:1: E402 module level import not at top of file
147:16: E126 continuation line over-indented for hanging indent
152:58: E203 whitespace before ','
210:15: E126 continuation line over-indented for hanging indent
213:1: E402 module level import not at top of file
247:17: E203 whitespace before ':'
248:22: E203 whitespace before ':'
249:20: E203 whitespace before ':'
250:18: E203 whitespace before ':'
251:24: E203 whitespace before ':'
252:22: E203 whitespace before ':'
253:19: E203 whitespace before ':'
254:20: E203 whitespace before ':'
255:23: E203 whitespace before ':'
256:18: E203 whitespace before ':'
257:17: E203 whitespace before ':'
258:19: E203 whitespace before ':'
259:20: E203 whitespace before ':'
260:21: E203 whitespace before ':'
261:19: E203 whitespace before ':'
262:20: E203 whitespace before ':'
263:17: E203 whitespace before ':'
264:24: E203 whitespace before ':'
265:23: E203 whitespace before ':'
266:21: E203 whitespace before ':'
267:37: E203 whitespace before ':'
281:4: E131 continuation line unaligned for hanging indent
310:1: W391 blank line at end of file


1: D205 1 blank line required between summary line and description (found 0)
1: D208 Docstring is over-indented
25:1: I100 Imports statements are in the wrong order. import logging should be before import sys
25:1: F401 'logging' imported but unused
27:1: I100 Imports statements are in the wrong order. import re should be before import urllib2
31:1: I101 Imported names are in the wrong order. Should be CFG_BIBINDEX_FULLTEXT_INDEX_LOCAL_FILES_ONLY, CFG_BIBINDEX_SPLASH_PAGES, CFG_SOLR_URL, CFG_XAPIAN_ENABLED
31:1: I100 Imports statements are in the wrong order. from invenio.config should be before from six
32:6: E126 continuation line over-indented for hanging indent
37:1: I100 Imports statements are in the wrong order. from invenio.legacy.miscutil.solrutils_bibindex_indexer should be before from invenio.utils.html
39:1: I101 Imported names are in the wrong order. Should be BibRecDocs, InvenioBibDocFileError, bibdocfile_url_p, bibdocfile_url_to_bibdoc, download_url
39:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from invenio.legacy.miscutil.xapianutils_bibindex_indexer
40:6: E126 continuation line over-indented for hanging indent
44:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.bibsched.bibtask
45:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.ext.logging
45:1: I201 Missing newline before sections or imports.
46:1: I201 Missing newline before sections or imports.
48:29: E261 at least two spaces before inline comment
51: D203 1 blank line required before class docstring (found 0)
51: D208 Docstring is over-indented
51: D200 One-line docstring should fit on one line with quotes (found 3)
56:41: E251 unexpected spaces around keyword / parameter equals
56:43: E251 unexpected spaces around keyword / parameter equals
56:66: E251 unexpected spaces around keyword / parameter equals
56:68: E251 unexpected spaces around keyword / parameter equals
56:94: E251 unexpected spaces around keyword / parameter equals
56:96: E251 unexpected spaces around keyword / parameter equals
56:123: E251 unexpected spaces around keyword / parameter equals
56:125: E251 unexpected spaces around keyword / parameter equals
56: D102 Missing docstring in public method
59:49: E127 continuation line over-indented for visual indent
60:49: E127 continuation line over-indented for visual indent
61:49: E127 continuation line over-indented for visual indent
63: D401 First line should be in imperative mood ('Allow', not 'Allows')
63: D400 First line should end with a period (not 'g')
67: D400 First line should end with a period (not 'r')
72:5: E303 too many blank lines (2)
72: D401 First line should be in imperative mood ('Return', not 'Returns')
72: D400 First line should end with a period (not 'y')
72: D205 1 blank line required between summary line and description (found 0)
72: D208 Docstring is over-indented
100:28: E713 test for membership should be 'not in'
152:66: E261 at least two spaces before inline comment
170:29: F821 undefined name 'file_converter_logger'
170:60: F821 undefined name 'old_logging_level'
179:5: E303 too many blank lines (2)
179: D102 Missing docstring in public method
183:5: E303 too many blank lines (2)
183: D102 Missing docstring in public method
186: D102 Missing docstring in public method
188:1: W391 blank line at end of file


4: L102 copyright year is outdated, expected 2015 but got 2012
30:1: I100 Imports statements are in the wrong order. import time should be before import urllib2
31:1: I100 Imports statements are in the wrong order. import os should be before import time
34:1: I100 Imports statements are in the wrong order. import tempfile should be before import urllib
35:1: I100 Imports statements are in the wrong order. import shutil should be before import tempfile
42:1: I101 Imported names are in the wrong order. Should be CFG_BIBUPLOAD_FFT_ALLOWED_LOCAL_PATHS, CFG_TMPSHAREDDIR
42:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.utils.url
42:1: E402 module level import not at top of file
49: D203 1 blank line required before class docstring (found 0)
49: D204 1 blank line required after class docstring (found 0)
51: D102 Missing docstring in public method
56: D203 1 blank line required before class docstring (found 0)
56: D204 1 blank line required after class docstring (found 0)
61: D400 First line should end with a period (not 'e')
61: D205 1 blank line required between summary line and description (found 0)
121: D400 First line should end with a period (not 'a')
121: D205 1 blank line required between summary line and description (found 0)
217: D401 First line should be in imperative mood ('Finalize', not 'Finalizes')
217: D400 First line should end with a period (not 't')
217: D205 1 blank line required between summary line and description (found 0)
252: D401 First line should be in imperative mood ('Copie', not 'Copies')
301: D209 Multi-line docstring closing quotes should be on a separate line
301: D400 First line should end with a period (not 't')
301: D205 1 blank line required between summary line and description (found 0)
323: D401 First line should be in imperative mood ('Open', not 'Opens')
323: D400 First line should end with a period (not 'd')
323: D205 1 blank line required between summary line and description (found 0)


This regex actually doesn't work.  Extensions students, for example, follow a different format; and if a student's last name has less than 4 characters it's also a different format.

Better is to just check if the variable `$user` appears in the output of `git log`.


indent


Lol... we also have such an always-false-IModelComparator. Great to see there is a wicket default now. :-D


The logic is good, but I dislike how the if statements get repeated twice.  The if statements beginning on lines 77 and 90 (of your version) have the exact same comments.  Combine the conditions together to reduce the code's redundancy.


This is the version I want.

Fix those two changes, then I'll merge.


`git archive` ?


@ders-noo it was implemented because I think it didn't work like that in some ruby version. It works fine for me in a recent version:

```ruby
RUBY_VERSION
# => "2.0.0"
{:verbose => true, :headers => { 'User-Agent' => "Fubar" }} ==  {:headers => { 'User-Agent' => "Fubar" }, :verbose => true }
# => true
```


O hack, good to see you are gone! :+1:


lol!


what?


Y esta propiedad? Qué esperabas conseguir? ```<content>```tiene otra función, agregar contenido por CSS

https://developer.mozilla.org/es/docs/Web/CSS/content


This completely breaks building in MSVC. "snprintf" is not defined.


I don' think we need this line, when we have the m_blocks.front() check


Alatis you done with studiou and stuff ?


慕安大大 您好:

                        我們是YUCSH的人。


Не вижу проблемы


Yes. I have one too since yesterday. Once Wicket 6.20.0 is released I'll get rid of it.


This will cause an issue for anyone using the filter as the argument count will change. What's best here? just leave an empty string?


Зачем передавать дробный размер?


@taconaut great work researching, might give the badger a try. It seems like it has dependency tracking too, maybe


// TODO: build()


This prevents us from using ALTCTL without GPS / flow.


By the way. @martin-g Do you know, whats the primary reason to have that comparator check in Component#setDefaultModelObject() ?
For me, explicitly setting the object, I naturally would expect the new object is set without any restrictions.
I often bypass that by using `getDefaultModel().setObject()`.
Thanx for feedback...


`*` might not be sufficient, but on the other hand `@dev` will enforce dev and thus might have the same side effect than `dev-master`.
I think we should better use branch aliases, since Composer best practice is not to use wildcards.


I hate concept to rename functions into "_xxx" where there are was "xxx" in VisualStudio :stuck_out_tongue_winking_eye:
But Jesus Christ invented macroses and I have to fix than trash at one line


with this commit reverted, there are quite a bit of things that are broken (we also dont have gcc etc..)


Hi @bjfr

this is super interesting, you should do a pull request on apache/tomee

can you do it?


// NEVER do


no. That's merge problem from yesterday, my bad. Let me fix it


Move to AbstractBuildableUnit


why you did not use EmailChanger service here, after validation?


¿Para que has puesto este ´´´position: relative´´´?


I presume we are no longer getting problems with the subtitles being a couple minutes out of sync.


git


Мне кажется, что это даже не стоит обсуждения док-блок можно всегда поправить. Мне так показалось логичнее.


Invalid generics


You don't want to refreshItemsFromTable() here because your table will be in Javascript.


Просто ширина/высота картинки де-факто должно быть целым числом. В данном случае, конечно может сойти и дробное, но в общем случае только целое.


No tiene mucho sentido usar ```display: inline-block```para después poner un ```width: 100%```. Hubieras sido mejor dejar el ```display: block``` por defecto que ya tiene el elemento.


I know, I'll try to find a solution. Like said elsewhere, I think the best would be to generalize branch alias usage.

> wouldn't it be more correct to specify you require symfony 2.4 or higher or something for this change? (well 2.6 considering the other change maybe)

How would it fix the problem? We still need ezpublish-kernel here.


When you run twice apt-get install odn-simple then you will have ip tables twice.

it will be looks like. IP tables lines are appended every postinstall run.
```
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8983 -j DROP
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8983 -j DROP
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8983 -j DROP
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8983 -j DROP
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8983 -j DROP
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -s 127.0.0.1/32 -p tcp -m tcp --dport 8983 -j ACCEPT
-A INPUT -p tcp -m tcp --dport 8983 -j DROP

```


![well done](http://cdn3.raywenderlich.com/wp-content/uploads/2012/09/Rage-well-done.png)


Меня больше волнует тот факт, что у нас ресайз производится очень криво =((( Я сейчас обновлю демо и покажу в чем проблема.


This means that Azure doesn't manage the sync, instead we have to. I think that is a disadvantage for Azure.


4: L102 copyright year is outdated, expected 2015 but got 2014


21:1: I101 Imported names are in the wrong order. Should be find_packages, setup
21:1: I201 Missing newline before sections or imports.


Have updated patch, using "memset(tbl_info.table, 0x00, TEE_MMU_UL1_SIZE);" , see "https://github.com/MrVan/optee_os/commit/3eff9058a653cd1dd47a482b73336cc447d865e0"


Building restore?


CBECC (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2143 of 2158 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.3%25-yellow.svg)


Il faut modifier l'Edito de notre partie ou pas ?


At a high level, anything that runs in an activity only runs when the user is interacting with the app. If you want to run things in the background, you need to run in a service. So if you were doing the azure option, you would call syncTable from ConfirmTripsAdapter.

Although if we have to do that, then it is not that much of a true background sync since we can just make the appropriate API calls anyway... :)


Qui dice cosi:
http://pyxb.sourceforge.net/PyXB-1.1.2/api/pyxb.binding.basis.simpleTypeDefinition-class.html#_InitializeFacetMap


heal() + protected onHeal() pattern


All of this should be javascript code...


Thanks for the quick fix! Maybe it worked because you didn't call ColorOut directly. The .onLoad function dealt properly with testTermForColorOut's output.


I think further adjustments would be nice to include in this PR.
No need to add config.h in every single file, but at least those
needing the GEOS_MVALUES define.

For output, I confirm cmake was the answer, thanks for the tip.
I think the relevant test is xmltester.

Trunk:
```
2/4 Test #2: xmltester ........................   Passed    6.22 sec
```

MValues:
```
2/4 Test #2: xmltester ........................   Passed    6.28 sec
```


慕安大大 您好:

                    我們是YUCSH的人。 我們有網站的問題想請教您，不知道方不方便請教您。
這是我們的E-mail:5002@yucsh.tp.edu.tw，非常希望你願意跟我們聯絡，我們也一直想跟你聯絡，只是
怕打擾到你，所以遲遲沒跟您聯絡，你近幾年的成就，讓YUCSH非常以您為榮。
ps: 為了跟你留言，只有註冊 github 留言，希望不會給你添麻煩。有空回來看看我們這些老人家以及學弟妹。


Hi Andri, thanks for your reply!

I was trying to use the exact same set of scripts, but had [an issue with getting those to work](https://github.com/fgrehm/vagrant-lxc-base-boxes/issues/35), however I managed to figure out the solution to that issue myself and it is working great now!


Sounds good. I was reverting a load of stuff to get things working again.


AsyncTasks are not true background execution. AsyncTasks are used to perform long running calls outside the UI thread, so that the screen is not frozen. But the AsyncTask won't run unless the user has launched the activity. If you want to do something that even when the user is not interacting with the phone, for example, you need to use a service, as described above.


might as well put these on the same import line


@snoyberg Would you review this patch which improves docs for the fun functions.


please make sure it has been removed from doc/python_reference.rst if it's in there


Fixed!


No it's not - it was broke when I changed this, and then it started working again!
There are more fundamental issues with it looking inside master/pct-data/ for the data.

Don't make statements that you're not sure are correct.


I think it is because of the notifications (`modelChanging()` and `modelChanged()`). These make no sense if the model object is actually the same.


lodashスクリプトからのコード生成ではなく、webpackを使ってrequireするようにしましょう。


s/An good/A good


converts?


Accidentally deleted my PR while cleaning up a merge conflict, so there's a new one at https://github.com/boundarydevices/imx_usb_loader/pull/21


post review catched by abdeali: commit message body is far too long. @fneu please break your commit messages in the future ok?


Found two typos, but otherwise a great improvement, thank you!


pls add below code.
if ("SIF" === network.key)
				network.meta.created = "2015-06-08T00:00:00.000000Z


@farky Please change ODCleanStore to UnifiedViews


yes it was literally this simple


http://stackoverflow.com/a/7392260
Instead of breaking stuff, first deprecate stuff?


Cross-Platform agnostic?


This looks promising, but it is again part of an activity. What happens when the activity is suspended, e.g. when the phone is locked? Will the continuous updates still happen?


That got lost in a previous update, thanks.


Hey @mamuz It does return ActionInterface


C'est vrai que cette correction n'a pas été appliquée en français alors qu'elle l'était en anglais, mais il manque alors aussi le renommage correspondant de l'image.


Posted some info on the forum here: http://forum.espruino.com/conversations/268197/#12325912


Hello
![_mg_3680-4000](https://cloud.githubusercontent.com/assets/8498997/8077138/b4230552-0f85-11e5-8c92-9b85ed1f64f7.jpg)


为


@ikesyo @mbogh I don't think it makes a difference. Constraints just have to be added to a common superview. The constraints are still applied to `nibView.superView = self.contentView`. But i agree, that for clarification, it should just say `self.contentView.addConstraints()`


no, you only need the travis cli tools with

 gem install travis -v 1.7.7 --no-rdoc --no-ri

and follow http://docs.travis-ci.com/user/deployment/pypi/
user and password are the pypi user and (encrypted) password

On 10 June 2015 at 06:06, Ian Lee <notifications@github.com> wrote:

> Not sure that I've made use of the deploy part of Travis before. Is this
> something that I should try getting an account for and trying independently?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cleder/fastkml/commit/fd84af85a87f1c32e684663ef4220c51833cba97#commitcomment-11603548>
> .
>

--
Best Regards,

Christian Ledermann

Newark-on-Trent - UK
Mobile : +44 7474997517

https://uk.linkedin.com/in/christianledermann
https://github.com/cleder/

<*)))>{

If you save the living environment, the biodiversity that we have left,
you will also automatically save the physical environment, too. But If
you only save the physical environment, you will ultimately lose both.

1) Don’t drive species to extinction

2) Don’t destroy a habitat that species rely on.

3) Don’t change the climate in ways that will result in the above.

}<(((*>


@plemarquand,

you should also add your name and fix to the header of `HttpTranslator.java`. I have also noticed that there are some commented out lines in there. Could you take a look at these and remove them if they are obsolete? I don't think it makes sense to keep unused code in the source files (that's what we have Git for). I promise, that this will be the last changes before I merge your PR ;-)


Sep


It's not there, but neither is plot_epochs. Should it be added?


Okay


LGTM


ok with it


probably, yeah


也


Are you using `user_login_failed` and  `_clean_credentials` somewhere?


This fixes #2.


ToDoFormData の変更は不要では？


Are you using `user_login_failed` and `_clean_credentials` somewhere in `views.py`?


Printing usage


style issue: please only import classes you need. (You can configure this in IntelliJ directly)


variable name (contestPhase1)


Extracting the ID, Species and formatting the output.


このへんに添付ファイルの削除、既読の削除を入れてください。


"this." is a code smell, please avoid


wrap in try-catch-finally. (to avoid deleting a rating if an error occurred in line 388-391)


I'm sorry, for me it get's dynamically broken virtually everywhere. Will shorten in the future.


delete this sentence; it's just a given that you'll be talking about these things, no need to point it out


delete this header


s/contain a match to a given pattern/match a given pattern


please include a description of how to set max lengths in confluence :) well done!


LGTM


I think it's is not a good idea to test implementation. The main idea od BDD is to test behaviours. conforming protocols in my opinion doesn't belong to behaviour.


aaaa


aaa


This paragraph is good, but it should be moved later in the writeup.


delete section header


s/and examine/to find


バックアップファイルもコミットされちゃっているので消しましょう！


hm, we compare to constant strings at multiple different lines here. Maybe we could substitute them by ENUMs, this way we could easily change their names in a refactoring whenever needed and the code wouldn't break here.. What do you think?


s/instances/lines

"instances" has a specific meaning in C++ that's different than the way you use it.


Thanks. I found one more typo. Fixed in 9de9805381ab05251822104db0038b312fc962e9.


modeで振り分けをするのであればinsert用のmodeも再定義する必要があるかもしれません。


From BDD point of view class is irrelevant


delete these two sentences; you're focus needs to be just on grep, not the alternatives


`Math.log()` gives the natural logarithm (to base `e`). We need log to base 2.
Also use `ceil()` instead of `round()` to round up.


Strange. I only added that method because php was complaining that should be implemented. Good that it is no longer needed!


This fixes #2


CSSで三角形を描く


That's because faction is defined on mobs, not mind: https://github.com/Baystation12/Baystation12/blob/master/code/modules/mob/mob_defines.dm#L164


this is not a proper fix for the issue.
it locks because you do `Popen.wait()` and `Popen.communicate()` at the same time.
just drop wait call and it will work fine


this WILL be the interim webserver, sitting ontop of a postgres server.
it will expose Web API for inserting raw data from the Scrapper/Parsing Server (and later on to small stores as well)
it will expose API for both getting raw data to the App Server, which will only take the data it needs (and it will expose our raw data to other groups/projects)


Actually I r a dumb, and mind has its own faction var.


This example is okay, but it's not super well motivated.  Why would I want to ever search for `cout`?  You need to tell me a story that convinces me what you're doing is actually really useful, and not just a cool trick.


This commit shouldn't be needed.


s/"cout"/`cout`

Anytime you're quoting something related to programming, use backticks


filenames are always in backticks


也


@ZuYuanZhou The Design is not satisfied, I may rewrite this when I get time. Actually I want to rewrite all the PNChart.


的


yes


@d0ugal thanks for taking a look. This is a simplification commit only (to be able to remove the docs_dir line from mkdocs.yml) and not needed to fix the actual issue.


Got "Fatal error: Allowed memory size of 1073741824 bytes exhausted (tried to allocate 32 bytes)" with a 1G memory_limit in php.ini whereas the other way around is ok for me.


This is a bad phrasing.  It doesn't explain the concept behind the example.  Better is:

> If you pass multiple files as command line parameters, grep will search through all of them.  For example:


delete the above two sentences


Better is:

> Both of these commands have the same output:

It's half the length and conveys the same information.


mocha is there anyway due to the use of grunt-mocha-istanbul


You need to explicitly say that grep takes input on stdin when no file is passed in


The above three sentences need to be simplified.


 adjust the code  BATCH_SIZE =4*10000;
 but the  spout get data from kestrelspeed is still  about 15000/s , I wanna  faster  at least 30000/s,
can u help me improve the speed?


Checked


Way too long of a sentence.  You can shorten this a lot.


The man page already tells me this in a more direct manner.  You need to provide an example that makes me *care*.  Your example just makes me say: "so what?"


@petrjasek @amagdas in the SuperdeskDataLayer, we are replacing the 'objectid' serializer with the one defined in the eve-elastic. The one that comes with the eve framework is fine. Any reason why we are replacing the serializer?


s/Or/or


Sure, that is fine then :)


Whelp, I've just grepped the code for the type datum/faction and it doesn't actually exist and removing the ```var/datum/faction/faction``` line doesn't cause any compilation errors.


zoekbalk erbij gemaakt en menu
menu moet nog overlegd worden met beheerpaneelgroep vanwege DRY methode

DRY?


link to regex tutorial


One more thing – should not `%032b` be changed to `%0128b` for IPv6?


Good point! i once had problems rebuilding the service layer when using class wildcards, such as presented here. for the sake of avoiding problems, Patrick's comment is very valid ;)


Again, why do I care?


Het kan toch worden hergebruikt daar?


ack


asd


ack from me


please use swift enumeration features:
`for index in 0..<sut.count`


space before `{`


> I figured it is probably not yet ready to be used by the public.

No, it's definitely ready for public use, we just don't have any good docs.

> configure the tests

Having a flick through the StyleCI/Config repo would probably help you out, particularly the tests. :)

The fixer names are the same as defined in PHP-CS-FIXER.


Done


Alright. You can change this IntelliJ settings here:
File -> Settings > Editor > Code Style > Java > Imports > Class count to use import with '*': 999


I have to agree. Try to archive the object to NSData and then back unarchive it.


We're already approaching 1000 repos enabled on StyleCI. :)


As-is, this gets a 20/50 on the technical section.  Everything you say is correct, but you don't convince me that it's actually useful.  Your example needs to be of a real world scenario that I might actually encounter.

As-is, this gets a 20/50 on the writing quality.  At a high level your writeup is well organized, but the individual sentences are way too wordy.  You need to spend a lot of time trying to simplify your explanations as much as possible.

You have until 3PM Thursday to fix these mistakes and get a higher grade.  If you would like more time, let me know and I'll see what I can do.


Shouldn't we just not validate files which don't have a file type? So validation would still work on old descriptions but we wouldn't show errors for old system descriptions which have links or directories.


These 2 lines above should add `s` to time value
```
keepalive_timeout 65s;
proxy_read_timeout 200s;
```
Not sure if Nginx will understand it as milliseconds.
I said this because I got 'Connection Timeout' error on Nginx but after I remove these 2 line (so it goes back to default value) , The error disappears.
http://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_read_timeout


delete


Shouldn't we also filter directories?


this file shouldn't be here


But that would mean that we disable file validation for the old descriptions completely, I don't think we want that.


`correclty` hmph.


Hi Mark,
thanks for the tip. Since we get the minimum required OS version as a string like "5.0" and not an API level from the [HockeyApp API](http://support.hockeyapp.net/kb/api/api-apps) we would then have to maintain a mapping of versions to API levels and update this with every new (even minor) release.


s/"article"/`article`


It is really not necessary here. Fixed.


Need a one sentence description on what a class is.


`command` is a typo


:) yeah done that one in haste :)  ,  thank you for correction


Typo detail.caretVisibe


Why do I care that it's called the preamble?  I don't, really, so just delete this.


Without pictures to see what these examples are doing, it's not super useful to me.


You need to tell me how to compile and build the documents using the `pdflatex` command.  Do that right here, not later.


What does "more specific" mean?  *You* have to be more specific!


Fixed in ef059da


it is reference, see bc9a06a2b0553d030a4ac1b229d830b9cbeb01b4


Just now installed Zookeeper. Next thing is nagios check wih check_zookeeper.py
What should be the value to -k argument? where can I find the key?
Thanks.


Should probably be something like "application/ldpatch+json" as we have a local solution here anyway


Shouldn't you copy `tcl-tk` formula into Linuxbrew?


What is that n for? :o


Awesome!


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/16042/testReport/) for this commit**


You need to follow the same rough format for how you present each example.  The format should be as follows:

brief introduction motivating the example
```
example latex code
```

detailed explanation of the example

Compile with:
```
$ pdflatex examplefile2.tex
```

img here

link to actual pdf here


Fixed!


You need to include the actual images and pdf files in your commit.  This is one of the few times when we want binary files in the repo.


I usually have the same order for constructor parameters and properties because I tend to order both in their order of use in the code
just sharing an little relevant habit..


index


No, this approach with limits is lame.

Insead we can create index of the view which would contain full paths of each line — so to get parent we could just look up value for certain line in index.
Brief tests show that it would be pretty fast — mark 12678 (4900 of which are children) items in a trice.

I’ll work on it this week.


Never talk about users in the third person.  The reader is the user.  Address them in the second person.  That is, say "You can also ..."


I don't really understand what you tried to do here, did you even test it? This is how it works:

If user checks any of these a string is appended to a list, then items in list are fetched from it and inserted into a final string
```
if (reasons.count() > 1)
    {
        int index = 0;
        while (index < (reasons.count() - 1))
        {
            this->UAAReportReason += reasons.at(index) + ", ";
            index++;
        }
        this->UAAReportReason = this->UAAReportReason.mid(0, this->UAAReportReason.length() - 2);
        this->UAAReportReason += " and " + reasons.at(index);
    } else if (reasons.count() == 1)
    {
        this->UAAReportReason += reasons.at(0) + ".";
    }
```
The code is ugly but clear. By adding extra spaces over the strings you just made the final string to contain double spaces.


would definitely have benefited from a specific commit here, would have documented the change in the history


You need to greatly simplify this whole paragraph because it's just a distraction at this point.


Контакты нужно научиться извлекать из таблицы, а не из аттрибутов чекбокса.
Нужно найти ряды, перебрать их в цикле - это есть
Потом в рядах нужно найти ячейки - этого нет
Из ячеек достать текст - фамилию имя телефон почту


Fixed in https://github.com/LionsAd/service_container/pull/41


You need to explicitly say that one of the main advantages of using Latex is the ability to write math formulas easily.


Yes.

For foldout sections, *everything* in the section has to be indented 2 spaces, on top of the indent they may already have. In these pages, that is basically everything except the beginning blurb and the headers themselves. If you look at the raw file in a monospaced font, it is more obvious. If you aren't sure if you have got it right, build the docs and check them! This is particularly useful for this page!


You use this sentence a lot.  You need to make it much shorter.


For all of these examples, delete the `Example:` and make the previous sentence introduce the example.


I set BEURK_INSTALL_DIR only in DEBUG_MODE. This can be an appropriate behavior that this option only work in "production mode". Maybe we need to specify this behavior in the beurk.conf file.


This is a sentence fragment.


To unblock the rest of the patch as I'd mixed up some obvious changes with something more contentious.

I'm happy to put -ffast-math back or send another CL.  Let me know!


FIxed in b8e3f8a


![logo_1718_b3b2](https://cloud.githubusercontent.com/assets/12825794/8077814/33d3f798-0f59-11e5-9584-4d0ac4bf91db.png)

a second logo. more colors been used too. will review logo.


Confirmed that there's a bug in ```createTables``` by using ```SQLiteTest.java``` and temporarily adding an extra createTable call just to see errors. I made edits to the ```createTables`` method in the latest commit to fix these issues.


Combine these into one example to be more efficient


Type and ref can not be null by `Component`'s contract
please make the Builder property final, add parameters to the `Builder` class (with early null check) and add parameters to the `builder()` method


This still doesn't use the ontology service


It doesn't "handle languages"; it "typesets code"


Zelfde CSS?


Again, the previous sentence should be introducing the example.


These examples all need to honour the indent above (as they are in a list, they should also be indented, otherwise they break out of the list (and also sometimes screw up the list mechanics)


Great stuff :) Now with Metal announced for Mac also I'm really interested in this as well. Any plans to get it fully working?


s/linux/Unix


making this comment make me realize that `DumbComponent`'s `getUuid()` and `getKey` methods do not comply with the `Component` interface contract (should never return `null` and throws Exception if so) on which Step my rely on


s/UNIX/Unix


Erwin DRY == Don't repeat yourself


You need to explicitly show me every command that needs to get run.  I should be able to just type whatever you say without thinking about it and have it work.  Never make me try to guess what command to use based on reading the text.


sfsdfdsf


Can we peg this to a specific revision?


I like the idea here, but adding all these other files to the repo is just a distraction.  Only add the tex file to the repo.


Streamlined use of `ProtocolError` in 53850083f5a921668421d96e711aec09168fa854. I am still not sure what are the best practices here, but the most consistent way seemed to qualify the error with the enum type (like `Error::ProtocolViolation`) to refer to protocol errors; though this still looks a bit cryptic (which `Error`?), and I would prefer something like `protocol::Error::ProtocolViolation`; but at this stage it starts looking like splitting hairs, and I would suggest we leave it as it is for now, and maybe have another look before going to main.


The need for this is detailed in the commit comment. What did it break for you?


You need to explicitly mention somewhere that one advantage of latex is that it uses plain text files, which makes it compatible with git.


1 sentence per line


yaml


You need to explicitly mention not adding pdf files; in this case, there are no executables created, so don't mention that.


:+1: nice


private methods should not come before public methods, in their order inside the class


Mention that this allows collaboration with other people using the same tools your already familiar with.


isClass = node.kind == cindex.CursorKind.CLASS_DECL


The top level header `#` should only be used on the main title of the document.


javascript  (?)


i'm going to attempt doing this using the syntax tree, without regex


js again?


delete this section and rephrase it as a complete sentence in the intro paragraph


ehhm.


ehhm ehhm.


It does, you don't really need to specify an ID.


link, but mention that this should come automatically installed with every latex distribution


js?


Thanks I'll have another look at it.


yaml


robustness principle (http://en.wikipedia.org/wiki/Robustness_principle) and convenience of unit tests


To avoid to add that import in every class where I use `UNAVAILABLE_ATTRIBUTE` or `NS_DESIGNATED_INITIALIZER`, I modified the `Podspec` file to import `CDTMacros` with [prefix_header_contents](https://github.com/cloudant/CDTDatastore/blob/feature-encryption/CDTDatastore.podspec#L48).


I haven't done the Deity changes yet - OA took me long enough that I just wanted to get the release out the door, so I squished the easy-fix bugs I knew about and released it. I did get the messages about creature skills, but again I haven't done anything with it yet. Next release is probably going to feature a fairly major overhaul in how creature info is stored anyway, so I'll do those changes at the same time.


It seems https://github.com/rollbar/error-stack-parser doesn't have any commit no? Maybe we can use the official repo?


delete this section; the ending of the last section is fine


Dette er testdata som er mer på linje med det som kommer fra ontology-service.


filter( x => x.2 != 0)


Even the cheapest receivers support HDMI input with multichannel PCM these days.


Images should never be jpg files.  Use png.


good
you can use "diff -u urfile myfile > patch" to generate a patch to apply them to ur source,like cherry pick,or if there isn't big difference just replace files.good luck ;)


Is the `if` block needed at all? It seems to me both branches assert the same values.


delete this image since you don't explain how to generate it


You've done a great job.  I think one last set of changes and you'll be good to get the extra credit.


I would like to see "mode" implemented as an property. Maybe it makes sense to expose mode in the inspector?


Good change.


Everything before this point distracts from your examples and should be deleted


:stuck_out_tongue_winking_eye: oh dear @Robinlovelace
It might not have been the commit which stopped it on your machine but it was the commit that stopped it on the server (as the data was in `pct-shiny/pct-data` on the server).  Please point the the more fundamental issues, or better yet fix them.


@tchap can't this action rollbacking be made more generic (ideally for any kind of action whatsoever), so that you don't have to duplicate code for it?


false


I don't agree. In this agree all parameters and returns could be nullable. For tests, if new QualityGate(name, emptyList()) is annoying for you, then you can add the constructor ```QualityGate(String name)```.


How can you discuss io redirection without mentioning stdin or stdout?


ClangModernize (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2174 of 2200 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.82%25-yellow.svg)


@rad3ks Paweł response (sorry for polish, but I'm to lazy to translate it)
>W 90% przypadków jeśli `describe`/`context` nie ma `beforeEach` to nie widzę sensu robić osobnego bloku.


You're introducing `>>` without ever having discussed it.


why is this here?  delete it


1: M111 unrecognized component name: legacy


Heh, I didn't mention you specifically @official, it's just a name of a java annotation that is being used in my code - java annotations are prefixed by a @ so github thought I was mentioning you.


sorry, I am a debian dummy, can you tell what the changes file is supposed to document and why it is important to have it as build result? Should it become part of the published repository or is it important for dependend builds?


You need a better title


You need to introduce the examples.


アドバイス，ありがとうございます．

コミットメッセージをもう少し詳しくします．

実は，Terminalでコミットせずに，まとめてGUIでやってしまったので，`_build`も追加してしまいました．次回からは，ちゃんとTerminalから必要なファイルだけ指定してコミットします．


s/"cpp"/`cpp`


I'm just curious why the latest data is not used all the time? This implementation will cause problems when the user starts composing a word and then removes it. Will cause the first char of that word to re-appear when the whole word is removed. When is the !/\u200b/.test(e.data) test needed which user input?


Template 000_config.py shouldn't have custom settings like this.
The placeholders can certainly be there...this seems the right place over templates/<>/config.py as it's a per-instance config, however the details shouldn't...they could be commented-out if they are not sensitive


I think this revision slightly improved Destroy All Humans, now i just get half a screen of more uniform crap, compared to the full screen of random crap :P


You should discuss the relevant principles of Unix at this point.


that is weird @devnook


34:1: I100 Imports statements are in the wrong order. from random should be before from invenio.utils.hash


El header no lleva ```margin``` por lo que no es necesario ponerlo a 0.


yaml


@tchap shouldn't RB submit the reviews when the release is staged? I was wondering about this recently -- GF used to close the reviews but SF doesn't seem to do it anymore.


31:1: I100 Imports statements are in the wrong order. from invenio.base.utils should be before import pkg_resources
42:1: E303 too many blank lines (4)
80:5: F401 'bibdocfile' imported but unused
83:5: E266 too many leading '#' for block comment


1: D100 Missing docstring in public module
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. import re should be before import sys
29:1: E402 module level import not at top of file
30:1: I100 Imports statements are in the wrong order. import os should be before import re
30:1: E402 module level import not at top of file
31:1: E402 module level import not at top of file
32:1: I100 Imports statements are in the wrong order. import fnmatch should be before import time
32:1: E402 module level import not at top of file
33:1: E402 module level import not at top of file
34:1: I101 Imported names are in the wrong order. Should be DEBUG, debug, getLogger
34:1: E402 module level import not at top of file
35:1: I101 Imported names are in the wrong order. Should be OptionGroup, OptionParser, OptionValueError
35:1: E402 module level import not at top of file
36:1: I201 Missing newline before sections or imports.
36:1: E402 module level import not at top of file
38:1: I100 Imports statements are in the wrong order. from invenio.base.factory should be before from six
38:1: E402 module level import not at top of file
40:1: E402 module level import not at top of file
41:1: I101 Imported names are in the wrong order. Should be CFG_BIBDOCFILE_FILEDIR, CFG_SITE_RECORD, CFG_SITE_URL, CFG_TMPSHAREDDIR
41:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.ext.logging
41:1: E402 module level import not at top of file
43:1: I101 Imported names are in the wrong order. Should be BibDoc, BibRecDocs, CFG_BIBDOCFILE_AVAILABLE_FLAGS, InvenioBibDocFileError, KEEP_OLD_VALUE, bibdocfile_url_to_bibdoc, check_valid_url, clean_url, decompose_bibdocfile_fullpath, decompose_bibdocfile_url, get_docname_from_url, guess_format_from_url, nice_size
43:1: E402 module level import not at top of file
49:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.bibdocfile.api
49:1: E402 module level import not at top of file
50:1: I201 Missing newline before sections or imports.
50:1: E402 module level import not at top of file
51:1: I101 Imported names are in the wrong order. Should be wait_for_user, wrap_text_in_a_box
51:1: E402 module level import not at top of file
52:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.utils.text
52:1: E402 module level import not at top of file
53:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibsched.bibtask should be before from invenio.legacy.dbquery
53:1: E402 module level import not at top of file
54:1: E402 module level import not at top of file
55:1: I100 Imports statements are in the wrong order. from invenio.utils.shell should be before from invenio.utils.text
55:1: E402 module level import not at top of file
57:1: E302 expected 2 blank lines, found 1
60:1: E302 expected 2 blank lines, found 1
66:1: E302 expected 2 blank lines, found 1
66: D209 Multi-line docstring closing quotes should be on a separate line
66: D400 First line should end with a period (not ',')
66: D205 1 blank line required between summary line and description (found 0)
86:1: E302 expected 2 blank lines, found 1
86: D400 First line should end with a period (not 'y')
86: D205 1 blank line required between summary line and description (found 0)
106:1: E302 expected 2 blank lines, found 0
106: D401 First line should be in imperative mood ('Return', not 'Returns')
106: D209 Multi-line docstring closing quotes should be on a separate line
106: D205 1 blank line required between summary line and description (found 0)
106: D208 Docstring is over-indented
113:19: E231 missing whitespace after ':'
113:32: E231 missing whitespace after ':'
113:42: E231 missing whitespace after ':'
113:50: E231 missing whitespace after ':'
126:1: E302 expected 2 blank lines, found 1
126: D401 First line should be in imperative mood ('Return', not 'Returns')
126: D400 First line should end with a period (not 'e')
129:19: E225 missing whitespace around operator
132:19: E225 missing whitespace around operator
138:1: E302 expected 2 blank lines, found 1
162:5: E303 too many blank lines (3)
201:1: E302 expected 2 blank lines, found 1
259:1: E302 expected 2 blank lines, found 1
259: D209 Multi-line docstring closing quotes should be on a separate line
259: D400 First line should end with a period (not 'd')
259: D205 1 blank line required between summary line and description (found 0)
290:1: E302 expected 2 blank lines, found 1
290: D209 Multi-line docstring closing quotes should be on a separate line
290: D202 No blank lines allowed after function docstring (found 1)
290: D400 First line should end with a period (not 'd')
290: D205 1 blank line required between summary line and description (found 0)
320:1: E302 expected 2 blank lines, found 1
326:28: W602 deprecated form of raising exception
328:28: W602 deprecated form of raising exception
330:1: E302 expected 2 blank lines, found 1
336:28: W602 deprecated form of raising exception
338:28: W602 deprecated form of raising exception
340:1: E302 expected 2 blank lines, found 1
340: D200 One-line docstring should fit on one line with quotes (found 3)
353:1: E302 expected 2 blank lines, found 1
360:1: E302 expected 2 blank lines, found 1
367:1: E302 expected 2 blank lines, found 1
374:1: E302 expected 2 blank lines, found 1
381:1: E302 expected 2 blank lines, found 1
381: D209 Multi-line docstring closing quotes should be on a separate line
381: D400 First line should end with a period (not 'd')
381: D205 1 blank line required between summary line and description (found 0)
392:1: E302 expected 2 blank lines, found 1
398:9: E266 too many leading '#' for block comment
404:1: E302 expected 2 blank lines, found 1
404: D209 Multi-line docstring closing quotes should be on a separate line
404: D205 1 blank line required between summary line and description (found 0)
416:1: E302 expected 2 blank lines, found 1
427:1: E302 expected 2 blank lines, found 1
427: D400 First line should end with a period (not 's')
442:248: E261 at least two spaces before inline comment
443:244: E261 at least two spaces before inline comment
444:206: E261 at least two spaces before inline comment
446:1: E302 expected 2 blank lines, found 1
446: D101 Missing docstring in public class
447: D102 Missing docstring in public method
454:1: E302 expected 2 blank lines, found 1
454: D202 No blank lines allowed after function docstring (found 1)
457: D400 First line should end with a period (not 'm')
457: D205 1 blank line required between summary line and description (found 0)
469: D301 Use r""" if any backslashes in a docstring
469: D209 Multi-line docstring closing quotes should be on a separate line
469: D400 First line should end with a period (not 'm')
469: D205 1 blank line required between summary line and description (found 0)
481:5: E128 continuation line under-indented for visual indent
482:9: E128 continuation line under-indented for visual indent
583:1: E302 expected 2 blank lines, found 1
587:1: E302 expected 2 blank lines, found 1
621:1: E302 expected 2 blank lines, found 1
627:24: E225 missing whitespace around operator
630:50: E261 at least two spaces before inline comment
637:1: E302 expected 2 blank lines, found 1
647:31: W602 deprecated form of raising exception
657:18: E203 whitespace before ':'
658:18: E203 whitespace before ':'
659:22: E203 whitespace before ':'
660:22: E203 whitespace before ':'
661:18: E203 whitespace before ':'
662:17: E203 whitespace before ':'
663:14: E203 whitespace before ':'
668:1: E302 expected 2 blank lines, found 1
680:31: W602 deprecated form of raising exception
686:18: E203 whitespace before ':'
687:18: E203 whitespace before ':'
688:22: E203 whitespace before ':'
689:18: E203 whitespace before ':'
690:22: E203 whitespace before ':'
691:22: E203 whitespace before ':'
692:18: E203 whitespace before ':'
693:17: E203 whitespace before ':'
694:14: E203 whitespace before ':'
695:18: E203 whitespace before ':'
699:1: E302 expected 2 blank lines, found 1
738:1: E302 expected 2 blank lines, found 1
764:1: E302 expected 2 blank lines, found 1
773:31: E261 at least two spaces before inline comment
774:18: E203 whitespace before ':'
774:32: E203 whitespace before ':'
774:57: E203 whitespace before ':'
777:1: E302 expected 2 blank lines, found 1
777: D400 First line should end with a period (not 'm')
803:1: E302 expected 2 blank lines, found 1
809:42: E203 whitespace before ':'
809:63: E203 whitespace before ':'
812:1: E302 expected 2 blank lines, found 1
817:38: E203 whitespace before ':'
820:38: E203 whitespace before ':'
823:1: E302 expected 2 blank lines, found 1
852:1: E302 expected 2 blank lines, found 1
869:1: E302 expected 2 blank lines, found 1
892:1: E302 expected 2 blank lines, found 1
909:1: E302 expected 2 blank lines, found 1
921:27: E203 whitespace before ':'
922:27: E203 whitespace before ':'
926:1: E302 expected 2 blank lines, found 1
938:5: B901 blind except: statement
939:31: W602 deprecated form of raising exception
940:18: E203 whitespace before ':'
940:32: E203 whitespace before ':'
940:53: E203 whitespace before ':'
940:73: E203 whitespace before ':'
940:96: E203 whitespace before ':'
943:1: E302 expected 2 blank lines, found 1
954:5: B901 blind except: statement
955:31: W602 deprecated form of raising exception
956:18: E203 whitespace before ':'
956:32: E203 whitespace before ':'
956:53: E203 whitespace before ':'
956:74: E203 whitespace before ':'
959:1: E302 expected 2 blank lines, found 1
959: D400 First line should end with a period (not 'e')
992:1: E302 expected 2 blank lines, found 1
1006:1: E302 expected 2 blank lines, found 1
1021:26: E203 whitespace before ':'
1022:26: E203 whitespace before ':'
1026:1: E302 expected 2 blank lines, found 1
1043:26: E203 whitespace before ':'
1044:26: E203 whitespace before ':'
1048:1: E302 expected 2 blank lines, found 1
1057:1: E302 expected 2 blank lines, found 1
1073:24: E201 whitespace after '('
1076:13: E128 continuation line under-indented for visual indent
1080:13: E128 continuation line under-indented for visual indent
1083:1: E302 expected 2 blank lines, found 1
1096:63: E203 whitespace before ','
1100:1: E302 expected 2 blank lines, found 1
1123:9: B901 blind except: statement
1124:35: W602 deprecated form of raising exception
1149:1: E302 expected 2 blank lines, found 1
1157:9: B901 blind except: statement
1158:35: W602 deprecated form of raising exception
1185: D103 Missing docstring in public function


1: D400 First line should end with a period (not 's')
1: D205 1 blank line required between summary line and description (found 0)
73:1: I100 Imports statements are in the wrong order. import os should be before from six.moves
73:1: I201 Missing newline before sections or imports.
75:1: I100 Imports statements are in the wrong order. import cgi should be before import time
79:1: I101 Imported names are in the wrong order. Should be CFG_CERN_SITE, CFG_SITE_LANG, CFG_SITE_RECORD, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_TMPSHAREDDIR, CFG_WEBSUBMIT_STORAGEDIR
80:6: E126 continuation line over-indented for hanging indent
87:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
89:1: I101 Imported names are in the wrong order. Should be BibDocFile, BibDocMoreInfo, BibRecDocs, InvenioBibDocFileError, calculate_md5, decompose_file
89:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from invenio.legacy.bibdocfile.cli
90:6: E126 continuation line over-indented for hanging indent
92:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.bibdocfile.api
95:1: I100 Imports statements are in the wrong order. from invenio.utils.html should be before from invenio.utils.url
96:1: I101 Imported names are in the wrong order. Should be bibtask_allocate_sequenceid, task_low_level_submission
96:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibsched.bibtask should be before from invenio.utils.html
96:1: F401 'task_low_level_submission' imported but unused
101:1: E302 expected 2 blank lines, found 1
101: D401 First line should be in imperative mood ('Return', not 'Returns')
437:31: E128 continuation line under-indented for visual indent
438:31: E128 continuation line under-indented for visual indent
439:31: E128 continuation line under-indented for visual indent
443:81: E502 the backslash is redundant between brackets
455:31: E128 continuation line under-indented for visual indent
456:31: E128 continuation line under-indented for visual indent
460:68: E502 the backslash is redundant between brackets
461:48: E502 the backslash is redundant between brackets
462:62: E502 the backslash is redundant between brackets
475:64: E502 the backslash is redundant between brackets
476:44: E502 the backslash is redundant between brackets
477:58: E502 the backslash is redundant between brackets
500:5: B901 blind except: statement
540:9: E266 too many leading '#' for block comment
544:10: E121 continuation line under-indented for hanging indent
545:10: E128 continuation line under-indented for visual indent
546:10: E128 continuation line under-indented for visual indent
547:10: E128 continuation line under-indented for visual indent
550:16: E127 continuation line over-indented for visual indent
551:16: E127 continuation line over-indented for visual indent
557:9: E266 too many leading '#' for block comment
558:9: E266 too many leading '#' for block comment
560:16: E127 continuation line over-indented for visual indent
560:74: E502 the backslash is redundant between brackets
561:46: E502 the backslash is redundant between brackets
562:67: E502 the backslash is redundant between brackets
564:49: E502 the backslash is redundant between brackets
565:74: E502 the backslash is redundant between brackets
589:22: E127 continuation line over-indented for visual indent
590:22: E127 continuation line over-indented for visual indent
591:22: E127 continuation line over-indented for visual indent
591:66: E502 the backslash is redundant between brackets
598:24: E127 continuation line over-indented for visual indent
604:34: E126 continuation line over-indented for hanging indent
607:45: E128 continuation line under-indented for visual indent
608:45: E128 continuation line under-indented for visual indent
613:24: E127 continuation line over-indented for visual indent
613:72: E502 the backslash is redundant between brackets
614:84: E502 the backslash is redundant between brackets
615:74: E502 the backslash is redundant between brackets
626:74: E502 the backslash is redundant between brackets
627:42: E127 continuation line over-indented for visual indent
636:26: E127 continuation line over-indented for visual indent
636:40: E502 the backslash is redundant between brackets
645:40: E272 multiple spaces before keyword
646:26: E127 continuation line over-indented for visual indent
651:28: E127 continuation line over-indented for visual indent
694:21: F841 local variable 'additional_formats' is assigned to but never used
713:28: E222 multiple spaces after operator
717:20: E126 continuation line over-indented for hanging indent
717:69: E502 the backslash is redundant between brackets
725:22: E127 continuation line over-indented for visual indent
739:9: E303 too many blank lines (2)
740:18: E127 continuation line over-indented for visual indent
740:67: E502 the backslash is redundant between brackets
750:5: E266 too many leading '#' for block comment
781:59: E502 the backslash is redundant between brackets
782:25: E128 continuation line under-indented for visual indent
782:68: E502 the backslash is redundant between brackets
787:91: E502 the backslash is redundant between brackets
788:74: E502 the backslash is redundant between brackets
789:63: E502 the backslash is redundant between brackets
790:60: E502 the backslash is redundant between brackets
801:25: E126 continuation line over-indented for hanging indent
801:95: E502 the backslash is redundant between brackets
802:74: E502 the backslash is redundant between brackets
803:67: E502 the backslash is redundant between brackets
845:16: E127 continuation line over-indented for visual indent
849:40: E231 missing whitespace after ':'
860:20: E126 continuation line over-indented for hanging indent
867:12: E127 continuation line over-indented for visual indent
886:9: E122 continuation line missing indentation or outdented
893:6: E122 continuation line missing indentation or outdented
898:112: E502 the backslash is redundant between brackets
918:1: E302 expected 2 blank lines, found 1
918: D401 First line should be in imperative mood ('Create', not 'Creates')
918: D400 First line should end with a period (not 'c')
981:90: E502 the backslash is redundant between brackets
992:41: E261 at least two spaces before inline comment
1000:12: E121 continuation line under-indented for hanging indent
1016:12: E127 continuation line over-indented for visual indent
1039:12: E127 continuation line over-indented for visual indent
1075:12: E127 continuation line over-indented for visual indent
1091:9: E122 continuation line missing indentation or outdented
1093:22: E231 missing whitespace after ':'
1099:1: E302 expected 2 blank lines, found 1
1099: D401 First line should be in imperative mood ('Parse', not 'Parses')
1099: D400 First line should end with a period (not 't')
1099: D205 1 blank line required between summary line and description (found 0)
1127:29: E261 at least two spaces before inline comment
1152:16: E127 continuation line over-indented for visual indent
1156:20: E127 continuation line over-indented for visual indent
1163:22: E127 continuation line over-indented for visual indent
1168:53: E251 unexpected spaces around keyword / parameter equals
1168:55: E251 unexpected spaces around keyword / parameter equals
1168:75: E251 unexpected spaces around keyword / parameter equals
1168:77: E251 unexpected spaces around keyword / parameter equals
1170:60: E231 missing whitespace after ':'
1172:56: E231 missing whitespace after ':'
1174:83: E231 missing whitespace after ','
1200:16: E127 continuation line over-indented for visual indent
1208:60: E231 missing whitespace after ':'
1210:56: E231 missing whitespace after ':'
1211:71: E502 the backslash is redundant between brackets
1220:1: E302 expected 2 blank lines, found 1
1220: D400 First line should end with a period (not 'k')
1232:1: E302 expected 2 blank lines, found 1
1232: D400 First line should end with a period (not 'k')
1245:1: E302 expected 2 blank lines, found 1
1245: D401 First line should be in imperative mood ('Log', not 'Logs')
1302:21: E221 multiple spaces before operator
1303:15: E127 continuation line over-indented for visual indent
1303:48: E221 multiple spaces before operator
1304:15: E127 continuation line over-indented for visual indent
1304:24: E221 multiple spaces before operator
1305:15: E127 continuation line over-indented for visual indent
1305:48: E221 multiple spaces before operator
1306:15: E127 continuation line over-indented for visual indent
1307:15: E127 continuation line over-indented for visual indent
1307:44: E221 multiple spaces before operator
1308:15: E127 continuation line over-indented for visual indent
1308:22: E221 multiple spaces before operator
1309:15: E127 continuation line over-indented for visual indent
1309:47: E221 multiple spaces before operator
1310:15: E127 continuation line over-indented for visual indent
1310:31: E221 multiple spaces before operator
1311:15: E127 continuation line over-indented for visual indent
1312:38: E225 missing whitespace around operator
1317:1: E302 expected 2 blank lines, found 1
1317: D401 First line should be in imperative mood ('Read', not 'Reads')
1317: D400 First line should end with a period (not 's')
1338:34: F841 local variable 'e' is assigned to but never used
1352:13: B901 blind except: statement
1359:69: E502 the backslash is redundant between brackets
1364:5: B901 blind except: statement
1369:1: E302 expected 2 blank lines, found 1
1369: D401 First line should be in imperative mood ('Return', not 'Returns')
1369: D400 First line should end with a period (not 'o')
1369: D205 1 blank line required between summary line and description (found 0)
1394:20: E128 continuation line under-indented for visual indent
1395:20: E128 continuation line under-indented for visual indent
1396:20: E128 continuation line under-indented for visual indent
1397:20: E128 continuation line under-indented for visual indent
1398:20: E128 continuation line under-indented for visual indent
1399:20: E128 continuation line under-indented for visual indent
1400:20: E128 continuation line under-indented for visual indent
1401:20: E128 continuation line under-indented for visual indent
1402:20: E128 continuation line under-indented for visual indent
1403:20: E128 continuation line under-indented for visual indent
1404:20: E128 continuation line under-indented for visual indent
1405:20: E128 continuation line under-indented for visual indent
1406:20: E128 continuation line under-indented for visual indent
1410:1: E302 expected 2 blank lines, found 1
1410: D400 First line should end with a period (not 's')
1410: D205 1 blank line required between summary line and description (found 0)
1421:67: E502 the backslash is redundant between brackets
1422:69: E502 the backslash is redundant between brackets
1422:66: E203 whitespace before ','
1423:83: E502 the backslash is redundant between brackets
1426:1: E302 expected 2 blank lines, found 1
1426: D400 First line should end with a period (not 'e')
1426: D205 1 blank line required between summary line and description (found 0)
1438:54: E502 the backslash is redundant between brackets
1445:1: E302 expected 2 blank lines, found 1
1445: D401 First line should be in imperative mood ('Return', not 'Returns')
1445: D202 No blank lines allowed after function docstring (found 1)
1445: D400 First line should end with a period (not 'c')
1445: D205 1 blank line required between summary line and description (found 0)
1457:59: E502 the backslash is redundant between brackets
1458:40: E502 the backslash is redundant between brackets
1463:56: E502 the backslash is redundant between brackets
1467:1: E302 expected 2 blank lines, found 1
1467: D401 First line should be in imperative mood ('Return', not 'Returns')
1467: D400 First line should end with a period (not 's')
1467: D205 1 blank line required between summary line and description (found 0)
1479:54: E502 the backslash is redundant between brackets
1480:20: E127 continuation line over-indented for visual indent
1485:1: E302 expected 2 blank lines, found 1
1485: D401 First line should be in imperative mood ('Return', not 'Returns')
1485: D400 First line should end with a period (not ',')
1485: D205 1 blank line required between summary line and description (found 0)
1501:69: E502 the backslash is redundant between brackets
1507:61: E502 the backslash is redundant between brackets
1515:1: E302 expected 2 blank lines, found 1
1515: D400 First line should end with a period (not ')')
1515: D205 1 blank line required between summary line and description (found 0)
1537:1: E302 expected 2 blank lines, found 1
1537: D401 First line should be in imperative mood ('Delete', not 'Deletes')
1537: D205 1 blank line required between summary line and description (found 0)
1548:42: E225 missing whitespace around operator
1551:1: E302 expected 2 blank lines, found 1
1551: D401 First line should be in imperative mood ('Washe', not 'Washes')
1551: D400 First line should end with a period (not 'e')
1551: D205 1 blank line required between summary line and description (found 0)
1656:12: E127 continuation line over-indented for visual indent
1657:46: E261 at least two spaces before inline comment
1658:47: E114 indentation is not a multiple of four (comment)
1658:47: E116 unexpected indentation (comment)
1664:46: E261 at least two spaces before inline comment
1682:12: E127 continuation line over-indented for visual indent
1687:12: E127 continuation line over-indented for visual indent
1690:16: E126 continuation line over-indented for hanging indent
1699:16: E126 continuation line over-indented for hanging indent
1708:40: E502 the backslash is redundant between brackets
1709:63: E502 the backslash is redundant between brackets
1710:37: E502 the backslash is redundant between brackets
1711:54: E502 the backslash is redundant between brackets
1719:40: E502 the backslash is redundant between brackets
1720:62: E502 the backslash is redundant between brackets
1721:37: E502 the backslash is redundant between brackets
1722:53: E502 the backslash is redundant between brackets
1730:39: E502 the backslash is redundant between brackets
1731:60: E502 the backslash is redundant between brackets
1732:45: E502 the backslash is redundant between brackets
1732:15: E272 multiple spaces before keyword
1733:36: E502 the backslash is redundant between brackets
1735:42: E261 at least two spaces before inline comment
1737:14: E127 continuation line over-indented for visual indent
1754:35: E126 continuation line over-indented for hanging indent
1768:12: E127 continuation line over-indented for visual indent
1769:12: E127 continuation line over-indented for visual indent
1774:13: B901 blind except: statement
1781:13: E266 too many leading '#' for block comment
1782:13: E266 too many leading '#' for block comment
1814: D400 First line should end with a period (not 'y')
1814: D205 1 blank line required between summary line and description (found 0)
1851:29: E261 at least two spaces before inline comment
1853:5: F841 local variable 'sequence_id' is assigned to but never used
1865:24: E126 continuation line over-indented for hanging indent
1888:24: E126 continuation line over-indented for hanging indent
1898:13: E303 too many blank lines (3)
1925:20: E127 continuation line over-indented for visual indent
1928:17: B901 blind except: statement
1929:81: E502 the backslash is redundant between brackets
1930:94: E502 the backslash is redundant between brackets
1940:51: E261 at least two spaces before inline comment
1951: D401 First line should be in imperative mood ('Add', not 'Adds')
1951: D400 First line should end with a period (not 'I')
1968:81: E502 the backslash is redundant between brackets
1977:57: E502 the backslash is redundant between brackets
1986:57: E502 the backslash is redundant between brackets
1991:58: E502 the backslash is redundant between brackets
1992:57: E502 the backslash is redundant between brackets
2002:38: F841 local variable 'e' is assigned to but never used
2005:69: E502 the backslash is redundant between brackets
2006:69: E502 the backslash is redundant between brackets
2007:59: E502 the backslash is redundant between brackets
2011:1: E302 expected 2 blank lines, found 1
2011: D401 First line should be in imperative mood ('Add', not 'Adds')
2011: D400 First line should end with a period (not 'I')
2011: D200 One-line docstring should fit on one line with quotes (found 3)
2026:29: E126 continuation line over-indented for hanging indent
2033:59: E502 the backslash is redundant between brackets
2043:38: F841 local variable 'e' is assigned to but never used
2046:69: E502 the backslash is redundant between brackets
2047:71: E502 the backslash is redundant between brackets
2048:55: E502 the backslash is redundant between brackets
2050:32: E127 continuation line over-indented for visual indent
2052:1: E302 expected 2 blank lines, found 1
2052: D401 First line should be in imperative mood ('Revise', not 'Revises')
2087:89: E502 the backslash is redundant between brackets
2093:66: E502 the backslash is redundant between brackets
2094:72: E502 the backslash is redundant between brackets
2102:33: E126 continuation line over-indented for hanging indent
2110:70: E502 the backslash is redundant between brackets
2122:29: E126 continuation line over-indented for hanging indent
2130:80: E502 the backslash is redundant between brackets
2148:89: E502 the backslash is redundant between brackets
2153:81: E502 the backslash is redundant between brackets
2154:69: E502 the backslash is redundant between brackets
2155:67: E502 the backslash is redundant between brackets
2163:29: E127 continuation line over-indented for visual indent
2169:24: E127 continuation line over-indented for visual indent
2183:64: E225 missing whitespace around operator
2183:70: E225 missing whitespace around operator
2191:57: E502 the backslash is redundant between brackets
2199:57: E502 the backslash is redundant between brackets
2204:58: E502 the backslash is redundant between brackets
2205:57: E502 the backslash is redundant between brackets
2216:69: E502 the backslash is redundant between brackets
2217:57: E502 the backslash is redundant between brackets
2218:55: E502 the backslash is redundant between brackets
2224:1: E302 expected 2 blank lines, found 1
2224: D401 First line should be in imperative mood ('Delete', not 'Deletes')
2224: D400 First line should end with a period (not 'c')
2224: D200 One-line docstring should fit on one line with quotes (found 3)
2250:73: E502 the backslash is redundant between brackets
2251:57: E502 the backslash is redundant between brackets
2252:59: E502 the backslash is redundant between brackets
2256:1: E302 expected 2 blank lines, found 1
2256: D205 1 blank line required between summary line and description (found 0)
2271:36: E225 missing whitespace around operator
2275: D401 First line should be in imperative mood ('Return', not 'Returns')
2275: D400 First line should end with a period (not 'e')
2275: D205 1 blank line required between summary line and description (found 0)
2317:8: E221 multiple spaces before operator
2318:9: E122 continuation line missing indentation or outdented
2319:9: E122 continuation line missing indentation or outdented
2651:1: E302 expected 2 blank lines, found 1
2651: D401 First line should be in imperative mood ('Return', not 'Returns')
2651: D200 One-line docstring should fit on one line with quotes (found 3)


1: D400 First line should end with a period (not ')')
2: L102 copyright year is outdated, expected 2015 but got 2014
26:1: F401 'InvenioBibDocFileError' imported but unused
27:1: F401 'run_sql' imported but unused
28:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.dbquery
28:1: F401 'register_exception' imported but unused
32:1: E302 expected 2 blank lines, found 1
32: D101 Missing docstring in public class
33: D400 First line should end with a period (not 'n')
33: D205 1 blank line required between summary line and description (found 0)
49: D200 One-line docstring should fit on one line with quotes (found 3)
58: D400 First line should end with a period (not 'n')
58: D205 1 blank line required between summary line and description (found 0)
92: D209 Multi-line docstring closing quotes should be on a separate line
92: D205 1 blank line required between summary line and description (found 0)
124:5: E301 expected 1 blank line, found 0
124: D102 Missing docstring in public method
127:1: E302 expected 2 blank lines, found 1
127: D103 Missing docstring in public function
128:105: E203 whitespace before ','
130:1: E302 expected 2 blank lines, found 1
130:70: E261 at least two spaces before inline comment
130: D103 Missing docstring in public function
131:33: E251 unexpected spaces around keyword / parameter equals
131:35: E251 unexpected spaces around keyword / parameter equals
133:35: E251 unexpected spaces around keyword / parameter equals
133:37: E251 unexpected spaces around keyword / parameter equals